Skip to content

Conversation

php4fan
Copy link
Contributor

@php4fan php4fan commented Aug 14, 2025

Fixes #4831

@Crell
Copy link
Contributor

Crell commented Oct 3, 2025

@jrfnl I think we can still merge this, despite your observation in #4831. Do you [dis]agree?

@jrfnl
Copy link
Contributor

jrfnl commented Oct 3, 2025

@Crell Not for me to decide. I'd recommend having a look at the complete param info. Might be that re-ordering the info and/or adding some emphasis in select places would help. Not sure duplicating info is the best way as it makes maintenance of the docs harder.

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.

Documentation unclear about the default order of $matches in preg_match_all()
3 participants