Skip to content

Conversation

Manishearth
Copy link
Member

Fixes #7032. The issue itself was already fixed, this is just the regression test from the issue.

Copy link
Member

@robertbastian robertbastian left a comment

Choose a reason for hiding this comment

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

this is not a reference year test

I'm fine with not checking this in, this is also covered by my exhaustive testing pr

@sffc sffc changed the title Fix shift left with overflow panic for Hijri Add test for shift left with overflow panic for Hijri Oct 15, 2025
Copy link
Member

@sffc sffc left a comment

Choose a reason for hiding this comment

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

Seems prudent to explicitly test the error report from Firefox.

I agree that reference_year.rs is not the right home.

@robertbastian
Copy link
Member

It's not an error report from Firefox

@Manishearth
Copy link
Member Author

@robertbastian how do we know that? Firefox has been finding and filing a fair number of bugs on this crate around panics and such. Firefox uses icu_calendar.

@robertbastian
Copy link
Member

@Manishearth
Copy link
Member Author

Ah, I see.

I don't have a strong need to merge this, I generally like closing filed issues with regression tests.

@Manishearth
Copy link
Member Author

Moved it.

@Manishearth Manishearth force-pushed the month-range-regression branch from b76e365 to 8aee658 Compare October 16, 2025 18:49
@Manishearth
Copy link
Member Author

This can be merged if people want, otherwise we should close this and the linked issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Panic on attempt to shift left with overflow in Hijri calculation

3 participants