-
Notifications
You must be signed in to change notification settings - Fork 8k
ext/calendar: jewishtojd overflow on year argument. #18849
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.
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
Conversation
Upper limit set to the 7th millenium (Messianic Age) in the jewish calendar, around 2239 year in the gregorian calendar.
|
upper limit can be changed (up to INT_MAX - 1), in practice I do not think there is a need to go too far in the future. |
|
Since |
|
🙋♂️ Can I kindly oppose this change? This change would be similar to imposing a 215 year limit on PHP DateTime, is there a known and agreed upon 215 year limit on lifetime of PHP usage? The Jewish calendar does not end at year 6000, the Messianic Age does not state that time ends at year 6000. I'm not too worried because given the previous 3500+ years of history, if this limitation is not reverted now, someone else will be there requesting this arbitrary limit to be removed in 215 years from now. That, or PHP might already be extinct. Looking forward continuing using PHP's jewish calendar functionality way past year 6000 - PHP will be 300 years old in year 6055 😁 |
exactly, you and I won't be here to see it anyway no need to lose sleep over it ;) |
|
What I am losing sleep over is
Not so fast there :) |
|
all jokes aside, here the bug in question. |
|
Fair enough I guess I do like the eschatological code decision 😁 big fan of the Messianic Age making its way into PHP source 👍 |
Upper limit set to the 7th millenium (Messianic Age) in the jewish calendar,
around 2239 year in the gregorian calendar.