Skip to content

#[newtype_oparg] derive macro #7489

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ShaharNaveh wants to merge 14 commits into from
Closed

#[newtype_oparg] derive macro #7489

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
e599382
Basic struct support
ShaharNaveh Mar 19, 2026
b60f972
Base enum support
ShaharNaveh Mar 23, 2026
936a079
Modify code for new macro
ShaharNaveh Mar 23, 2026
94d66dd
No need for full feature
ShaharNaveh Mar 23, 2026
f5c2f9b
Remove debug code
ShaharNaveh Mar 23, 2026
9fbc6d3
Impl hash
ShaharNaveh Mar 23, 2026
49d2fb6
Merge remote-tracking branch 'upstream/main' into macro-newtype-oparg
ShaharNaveh Mar 24, 2026
f7b11b6
syn full feature
ShaharNaveh Mar 24, 2026
f21d855
Remove unwrap
ShaharNaveh Mar 24, 2026
77eae15
Use `TryFrom`
ShaharNaveh Mar 24, 2026
4d6c725
Convert `ConvertValueOparg` to new macro
ShaharNaveh Mar 24, 2026
c702fbe
Fix bug where `catch_all` is not the last element
ShaharNaveh Mar 24, 2026
796b61c
Don't match on 255
ShaharNaveh Mar 24, 2026
9691476
Validate `catch_all` variant drfinition
ShaharNaveh Mar 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Don't match on 255
  • Loading branch information
@ShaharNaveh
ShaharNaveh committed Mar 24, 2026
commit 796b61c955701a55f9a6f8ad2b85d0ce6d462def
3 changes: 1 addition & 2 deletions crates/vm/src/stdlib/_ast/other.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,8 @@ impl Node for ast::ConversionFlag {
object: PyObjectRef,
) -> PyResult<Self> {
// Python's AST uses ASCII codes: 's', 'r', 'a', -1=None
// Note: 255 is -1i8 as u8 (ruff's ConversionFlag::None)
match i32::try_from_object(vm, object)? {
-1 | 255 => Ok(Self::None),
-1 => Ok(Self::None),
x if x == b's' as i32 => Ok(Self::Str),
x if x == b'r' as i32 => Ok(Self::Repr),
x if x == b'a' as i32 => Ok(Self::Ascii),
Expand Down
Loading