Skip to content

Conversation

@mjansenDatabay
Copy link
Contributor

This function has no effect.
Prior to PHP 8.0.0, this function was used to
close the resource.

This function has no effect.
Prior to PHP 8.0.0, this function was used to
close the resource.
@mjansenDatabay mjansenDatabay added improvement php Pull requests that update Php code labels Jan 26, 2026
@mjansenDatabay mjansenDatabay changed the title Remove xml_parser_free from codebase Remove xml_parser_free from codebase (PHP 8.5 issue) Jan 26, 2026
Copy link
Member

@chfsx chfsx left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

Copy link
Contributor

@kergomard kergomard left a comment

Choose a reason for hiding this comment

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

Thank you very much @mjansenDatabay !

I approve the changes in the Test.

Best,
@kergomard

Copy link
Contributor

@dsstrassner dsstrassner left a comment

Choose a reason for hiding this comment

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

I follow @kergomard. 🦘

Copy link
Contributor

@thibsy thibsy left a comment

Choose a reason for hiding this comment

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

LGTM, thx!

Copy link
Contributor

@GitHamo GitHamo left a comment

Choose a reason for hiding this comment

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

Looks good, thanks! 👍

Copy link
Contributor

@sKarki999 sKarki999 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks a lot, @mjansenDatabay :)

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

Labels

improvement php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants