Skip to content

Pdo sqlite refactor#18804

Closed
devnexen wants to merge 1 commit intophp:masterfrom
devnexen:pdo_sqlite_refactor
Closed

Pdo sqlite refactor#18804
devnexen wants to merge 1 commit intophp:masterfrom
devnexen:pdo_sqlite_refactor

Conversation

@devnexen
Copy link
Copy Markdown
Member

@devnexen devnexen commented Jun 8, 2025

No description provided.

@devnexen devnexen marked this pull request as ready for review June 8, 2025 13:45
Comment thread ext/pdo_sqlite/sqlite_statement.c Outdated
Comment thread build/php.m4 Outdated
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.

Strictly speaking this requires an upgrading entry.

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.

Similarly, your newly added constants need upgrading entries

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.

This uses the old-style constant instead of the new one you introduced. Shouldn't both be tested?

Comment thread ext/pdo_sqlite/pdo_sqlite.c Outdated
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.

These are old-style constants instead of the subclass specific ones. Do we still want to add new ones to here instead of only continuing to evolve the subclasses?

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.

Yes, we should only add new ones on the subclasses. Reminds me that I wanted to slate the overloaded PDO instances with methods for deprecation in 8.5

Comment thread UPGRADING Outdated
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.

This is a class constant, not a global constant.

@devnexen devnexen force-pushed the pdo_sqlite_refactor branch from afcf17f to 7f80836 Compare June 9, 2025 12:22
allow to check if a statement is still running before reusage.

close phpGH-18804
@devnexen devnexen force-pushed the pdo_sqlite_refactor branch from 7f80836 to 55a9b03 Compare June 9, 2025 12:22
@devnexen devnexen closed this in 53231a8 Jun 9, 2025
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