Skip to content

ext/pdo_pgsql: Delete unused constants #18358

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

vrana
Copy link
Contributor

@vrana vrana commented Apr 19, 2025

These constants were added by 6ed1819 but they are not used anymore.

They are undocumented which is why I've stumbled upon this.

@devnexen
Copy link
Member

devnexen commented Apr 19, 2025

If that s the case, you need to remove the entries in the related stub too and regenerate using build/gen_stub.php

@devnexen devnexen changed the title Delete unused constants ext/pdo_pgsql: Delete unused constants Apr 19, 2025
@vrana vrana requested a review from kocsismate as a code owner April 19, 2025 21:12
@devnexen
Copy link
Member

Would be good to have a NEWS and above all an UPGRADING entry.

These constants were added by 6ed1819 but they are not used anymore.

They are undocumented which is why I've stumbled upon this.
Copy link
Member

@devnexen devnexen left a comment

Choose a reason for hiding this comment

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

LCTM @SakiTakamachi please have a look

Copy link
Member

@SakiTakamachi SakiTakamachi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

Just to be sure, I did a GitHub code search, but it seems there is no (public) code that uses these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants