Some improvements to holiday authority code (PR #25788)

[Lauri Nurmi]

Such as:

• Move implementations to .cpp
• Check valid date range for overridden implementations too

If something doesn't look like necessary, feel free to cherry-pick only the useful ones.

---

You can view, comment on, or merge this pull request online at:

  https://github.com/wxWidgets/wxWidgets/pull/25788

Commit Summary

• 68b27a4 Do not duplicate DoGetHolidaysInRange() in the derived class
• 83d7799 Move holiday implementations away from the header
• 907b3fe Move range check to the public GetHolidaysInRange()
• 62f7e66 Assert that DoGetHolidaysInRange and DoIsHoliday agree on holidays

File Changes

(2 files)

• M include/wx/datetime.h (50)
• M src/common/datetime.cpp (84)

Patch Links:

• https://github.com/wxWidgets/wxWidgets/pull/25788.patch
• https://github.com/wxWidgets/wxWidgets/pull/25788.diff

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/25788@github.com>

[VZ]

Merged #25788 into master.

—
Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/25788/issue_event/19677314997@github.com>