Skip to content

chore[fuzzer]: add u256 to implemented Int for i256 correctly#3576

Merged
joseph-isaacs merged 2 commits intodevelopfrom
ji/fix-decimal-int-range
Jun 19, 2025
Merged

chore[fuzzer]: add u256 to implemented Int for i256 correctly#3576
joseph-isaacs merged 2 commits intodevelopfrom
ji/fix-decimal-int-range

Conversation

@joseph-isaacs
Copy link
Contributor

No description provided.

Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
@joseph-isaacs joseph-isaacs enabled auto-merge (squash) June 19, 2025 20:50
@joseph-isaacs joseph-isaacs requested a review from robert3005 June 19, 2025 20:50
@github-actions github-actions bot added the changelog/chore A trivial change label Jun 19, 2025
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
Copy link
Contributor

@robert3005 robert3005 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And I thought I could just fake it with i256

@joseph-isaacs joseph-isaacs merged commit 873849f into develop Jun 19, 2025
33 checks passed
@joseph-isaacs joseph-isaacs deleted the ji/fix-decimal-int-range branch June 19, 2025 21:03
@joseph-isaacs
Copy link
Contributor Author

I think it can lead to a negative range len which is added to a start producing an empty range

mwlon pushed a commit to mwlon/vortex that referenced this pull request Jul 4, 2025
…ex-data#3576)

Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
Signed-off-by: mwlon <m.w.loncaric@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants