Skip to content

intl: document the procedural style of IntlGregorianCalendar methods - #5619

Open
lacatoire wants to merge 1 commit into
php:masterfrom
lacatoire:doc/intlgregcal-procedural
Open

intl: document the procedural style of IntlGregorianCalendar methods#5619
lacatoire wants to merge 1 commit into
php:masterfrom
lacatoire:doc/intlgregcal-procedural

Conversation

@lacatoire

Copy link
Copy Markdown
Member

The procedural functions intlgregcal_get_gregorian_change, intlgregcal_is_leap_year and intlgregcal_set_gregorian_change were undocumented. This adds their procedural synopsis to the existing OO method pages (mirroring the IntlCalendar convention), fills the short descriptions, and drops the &warn.undocumented.func; placeholders.

The deprecated intlgregcal_create_instance is intentionally left out.

@lacatoire
lacatoire force-pushed the doc/intlgregcal-procedural branch 2 times, most recently from 1b16799 to 4f8ae13 Compare June 16, 2026 11:58
@lacatoire
lacatoire force-pushed the doc/intlgregcal-procedural branch from 4f8ae13 to 9be3167 Compare June 23, 2026 14:07
</methodsynopsis>
<simpara>
Gets the Gregorian Calendar change date. This is the point at which the
calendar switches from Julian dates to Gregorian dates, expressed as a UNIX

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
calendar switches from Julian dates to Gregorian dates, expressed as a UNIX
calendar switches from Julian dates to Gregorian dates, expressed as a Unix

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's just Unix ;-)

</methodsynopsis>
<simpara>
Sets the Gregorian Calendar change date. This is the point at which the
calendar switches from Julian dates to Gregorian dates, expressed as a UNIX

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
calendar switches from Julian dates to Gregorian dates, expressed as a UNIX
calendar switches from Julian dates to Gregorian dates, expressed as a Unix


</para>
<simpara>
The Gregorian Calendar change date, expressed as a UNIX timestamp in

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
The Gregorian Calendar change date, expressed as a UNIX timestamp in
The Gregorian Calendar change date, expressed as a Unix timestamp in

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.

2 participants