Skip to content

Add optional native parser routing#385

Draft
adamziel wants to merge 1 commit intocodex/native-parser-php-facadefrom
codex/native-parser-class-routing
Draft

Add optional native parser routing#385
adamziel wants to merge 1 commit intocodex/native-parser-php-facadefrom
codex/native-parser-class-routing

Conversation

@adamziel
Copy link
Copy Markdown
Collaborator

Summary

  • add fallback WP_MySQL_Native_* classes that extend the PHP polyfills
  • route public MySQL lexer/parser classes through native classes when the extension preloads them
  • add the minimal PHP bridge that exports grammar data for native parser setup

Testing

  • php -l on changed PHP files
  • parser smoke: lex and parse SELECT 1 through WP_MySQL_Lexer and WP_MySQL_Parser

This PR is intentionally stacked after the polyfill extraction PR so the native routing diff is small and reviewable.

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.

1 participant