Skip to content
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

Jump to bottom

Fix msg_date_d and msg_date_e wrongly parsing dates after 29 Feb. Mon returned by month_d() is in 0..11 range. Fix unit-tests that were assuming 25 Mar 2004 to be Wednesday but it's actually Thursday. #250

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Fix msg_date_d and msg_date_e wrongly parsing dates after 29 Feb. Mon returned by month_d() is in 0..11 range. Fix unit-tests that were assuming 25 Mar 2004 to be Wednesday but it's actually Thursday. #250

merged 1 commit into from
Mar 1, 2024

Conversation

andywolk
Copy link
Member

@andywolk andywolk commented Mar 1, 2024

No description provided.

@andywolk andywolk added the bug Something isn't working label Mar 1, 2024
@andywolk andywolk requested review from mjerris and anthmFS March 1, 2024 17:24
@freeswitch-ci-system
Copy link

Unit-tests failed: https://public-artifacts.signalwire.cloud/drone/freeswitch/sofia-sip/522/artifacts.html

@andywolk andywolk changed the title Fix msg_date_d wrongly parsing dates after 29 feb. Mon returned by month_d() is in 0..11 range. Fix msg_date_d and msg_date_e wrongly parsing dates after 29 Feb. Mon returned by month_d() is in 0..11 range. Fix unit-tests that were assuming 25 Mar 2004 to be Wednesday but it's actually Thursday. Mar 1, 2024
@freeswitch-ci-system
Copy link

Unit-tests failed: https://public-artifacts.signalwire.cloud/drone/freeswitch/sofia-sip/523/artifacts.html

@andywolk
Fix msg_date_d and msg_date_e wrongly parsing dates after 29 Feb. Mon…
17aaa40 
… returned by month_d() is in 0..11 range. Fix unit-tests that were assuming 25 Mar 2004 to be Wednesday but it's actually Thursday.
@andywolk andywolk merged commit 50a509b into master Mar 1, 2024
18 checks passed
@andywolk andywolk deleted the leap branch March 1, 2024 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anthmFS anthmFS anthmFS approved these changes

@mjerris mjerris Awaiting requested review from mjerris

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andywolk @anthmFS