date: fix strftime O/E modifier parsing in C locale#11802
date: fix strftime O/E modifier parsing in C locale#11802Jah-yee wants to merge 1 commit intouutils:mainfrom
Conversation
In C locale, the O (alternative numeric symbols) and E (alternative format) modifiers should be ignored/stripped. However, they were being passed through as literal text because they weren't recognized as flags. This fix adds O and E to the recognized flag characters in parse_format_spec(), so they're properly handled as modifiers and stripped in C locale (as expected by GNU date). Fixes: uutils#11656
|
GNU testsuite comparison: |
This comment was marked as outdated.
This comment was marked as outdated.
|
@Jah-yee Thank you for your contribution, and welcome to the repo! I'm really sorry to say that I'm going to approve #11834 instead of this one, because the fixes are more complete (notice that if you cherry-pick both test-adding commits on that PR onto your branch, the |
No problem at all~Thx for your kind words and for your review! I will go ahead and close this. |
Good day,
Thank you for your work on this project. I hope this small fix is helpful.
Please let me know if there's anything to adjust.
Warmly, RoomWithOutRoof