[RFC] Deprecate xml_parser_free() (#19449)

DanielEScherzer · web-flow · commit 4de2ec3895e9 · 2025-08-12T05:44:32.000-07:00
https://wiki.php.net/rfc/deprecations_php_8_5
diff --git a/Zend/tests/bug34617.phpt b/Zend/tests/bug34617.phpt
@@ -11,7 +11,6 @@ function boom()
     $thing = new Thing();
     xml_set_object($reader, $thing);
     die("ok\n");
-    xml_parser_free($reader);
 }
 boom();
 ?>
diff --git a/ext/libxml/tests/libxml_entity_loading_disabled_by_default.phpt b/ext/libxml/tests/libxml_entity_loading_disabled_by_default.phpt
@@ -30,7 +30,6 @@ function parseXML2($xml) {
 function parseXML3($xml) {
   $p = xml_parser_create();
   xml_parse_into_struct($p, $xml, $vals, $index);
-  xml_parser_free($p);
   return var_export($vals, true);
 }
 
diff --git a/ext/xml/tests/bug25666.phpt b/ext/xml/tests/bug25666.phpt
@@ -28,7 +28,6 @@ $parser = xml_parser_create_ns("ISO-8859-1","@");
 xml_set_element_handler($parser,'start_elem','end_elem');
 xml_parser_set_option($parser,XML_OPTION_CASE_FOLDING,0);
 xml_parse($parser, $xml);
-xml_parser_free($parser);
 ?>
 --EXPECT--
 string(24) "http://example.com/foo@a"
diff --git a/ext/xml/tests/bug26528.phpt b/ext/xml/tests/bug26528.phpt
@@ -7,7 +7,6 @@ xml
     $sample = "<?xml version=\"1.0\"?><test attr=\"angle&lt;bracket\"/>";
     $parser = xml_parser_create();
     $res = xml_parse_into_struct($parser,$sample,$vals,$index);
-    xml_parser_free($parser);
     var_dump($vals);
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/bug26614.inc b/ext/xml/tests/bug26614.inc
@@ -69,5 +69,4 @@ foreach ($xmls as $desc => $xml) {
     xml_set_character_data_handler($xml_parser, "characterData");
     if (!xml_parse($xml_parser, $xml, true))
         echo "Error: ".xml_error_string(xml_get_error_code($xml_parser))."\n";
-    xml_parser_free($xml_parser);
 }
diff --git a/ext/xml/tests/bug27908.phpt b/ext/xml/tests/bug27908.phpt
@@ -12,7 +12,6 @@ function x_default_handler($xp,$data)
 $xp = xml_parser_create();
 xml_set_default_handler($xp,'x_default_handler');
 xml_parse($xp, '<root></root>',TRUE);
-xml_parser_free($xp);
 echo "Done\n";
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/bug30875.phpt b/ext/xml/tests/bug30875.phpt
@@ -14,7 +14,6 @@ XML;
 
 $parser = xml_parser_create();
 xml_parse_into_struct($parser, $xml, $vals);
-xml_parser_free($parser);
 var_dump($vals);
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/bug35447.phpt b/ext/xml/tests/bug35447.phpt
@@ -24,7 +24,6 @@ END_OF_XML;
 $parser = xml_parser_create_ns('UTF-8');
 xml_parser_set_option($parser,XML_OPTION_CASE_FOLDING,0);
 $result = xml_parse_into_struct($parser, $data, $vals, $index);
-xml_parser_free($parser);
 var_dump($vals);
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/bug46699.phpt b/ext/xml/tests/bug46699.phpt
@@ -25,7 +25,6 @@ $parser = xml_parser_create_ns("ISO-8859-1","@");
 xml_set_default_handler($parser,'defaultfunc');
 xml_parser_set_option($parser,XML_OPTION_CASE_FOLDING,0);
 xml_parse($parser, $xml);
-xml_parser_free($parser);
 ?>
 --EXPECTF--
 <a xmlns="http://example.com/foo"%axmlns:bar="http://example.com/bar">
diff --git a/ext/xml/tests/bug50576.phpt b/ext/xml/tests/bug50576.phpt
@@ -21,15 +21,13 @@ echo 'Index array' . PHP_EOL;
 print_r($index);
 echo 'Vals array' . PHP_EOL;
 print_r($vals);
-xml_parser_free($xml_parser);
 
 function startElement($parser, $name, $attribs) { echo $name . PHP_EOL; }
 function endElement($parser, $name) { echo $name . PHP_EOL; }
 $xml_parser = xml_parser_create();
 xml_set_element_handler($xml_parser, 'startElement', 'endElement');
 xml_parser_set_option($xml_parser, XML_OPTION_SKIP_TAGSTART, 4);
 xml_parse($xml_parser, $XML);
-xml_parser_free($xml_parser);
 
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/bug72714.phpt b/ext/xml/tests/bug72714.phpt
@@ -22,8 +22,6 @@ function parse($tagstart) {
 
     xml_parser_set_option($xml_parser, XML_OPTION_SKIP_TAGSTART, $tagstart);
     xml_parse($xml_parser, $xml);
-
-    xml_parser_free($xml_parser);
 }
 
 parse(3015809298423721);
diff --git a/ext/xml/tests/bug72793.phpt b/ext/xml/tests/bug72793.phpt
@@ -33,4 +33,6 @@ $xml_parser->free();
 ===DONE===
 --EXPECTF--
 Deprecated: Function xml_set_object() is deprecated since 8.4, provide a proper method callable to xml_set_*_handler() functions in %s on line %d
+
+Deprecated: Function xml_parser_free() is deprecated since 8.5, as it has no effect since PHP 8.0 in %s on line %d
 ===DONE===
diff --git a/ext/xml/tests/bug76874.phpt b/ext/xml/tests/bug76874.phpt
@@ -27,5 +27,6 @@ $object->test();
 
 ?>
 ===DONE===
---EXPECT--
+--EXPECTF--
+Deprecated: Function xml_parser_free() is deprecated since 8.5, as it has no effect since PHP 8.0 in %s on line %d
 ===DONE===
diff --git a/ext/xml/tests/set_element_handler_trampoline.phpt b/ext/xml/tests/set_element_handler_trampoline.phpt
@@ -42,21 +42,18 @@ echo "Both handlers are trampolines:\n";
 $parser = xml_parser_create();
 xml_set_element_handler($parser, $startCallback, $endCallback);
 xml_parse($parser, $xml, true);
-xml_parser_free($parser);
 
 echo "\nStart handler is trampoline, end handler method string:\n";
 $parser = xml_parser_create();
 xml_set_object($parser, $customParser);
 xml_set_element_handler($parser, $startCallback, 'endHandler');
 xml_parse($parser, $xml, true);
-xml_parser_free($parser);
 
 echo "\nEnd handler is trampoline, start handler method string:\n";
 $parser = xml_parser_create();
 xml_set_object($parser, $customParser);
 xml_set_element_handler($parser, 'startHandler', $endCallback);
 xml_parse($parser, $xml, true);
-xml_parser_free($parser);
 
 ?>
 --EXPECTF--
diff --git a/ext/xml/tests/set_element_handler_trampoline_errors.phpt b/ext/xml/tests/set_element_handler_trampoline_errors.phpt
@@ -36,7 +36,6 @@ try {
 } catch (\Throwable $e) {
     echo $e::class, ': ', $e->getMessage(), PHP_EOL;
 }
-xml_parser_free($parser);
 
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/set_handler_trampoline.phpt b/ext/xml/tests/set_handler_trampoline.phpt
@@ -24,7 +24,6 @@ HERE;
 $parser = xml_parser_create();
 xml_set_processing_instruction_handler($parser, $callback);
 xml_parse($parser, $xml, true);
-xml_parser_free($parser);
 
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/xml001.phpt b/ext/xml/tests/xml001.phpt
@@ -30,7 +30,6 @@ while ($data = fread($fp, 4096)) {
     }
 }
 print "parse complete\n";
-xml_parser_free($xml_parser);
 
 function startElement($parser, $name, $attribs)
 {
diff --git a/ext/xml/tests/xml002.phpt b/ext/xml/tests/xml002.phpt
@@ -72,7 +72,6 @@ while ($data = fread($fp, 4096)) {
     }
 }
 print "parse complete\n";
-xml_parser_free($xml_parser);
 
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/xml003.phpt b/ext/xml/tests/xml003.phpt
@@ -71,7 +71,6 @@ while ($data = fread($fp, 4096)) {
     }
 }
 print "parse complete\n";
-xml_parser_free($xml_parser);
 ?>
 --EXPECT--
 {?[<?xml version="1.0" encoding="ISO-8859-1"?>]}{?[
diff --git a/ext/xml/tests/xml004.phpt b/ext/xml/tests/xml004.phpt
@@ -13,15 +13,13 @@ $fp = fopen("xmltest.xml", "r");
 while ($data = fread($fp, 4096)) {
     xml_parse($xp, $data, feof($fp));
 }
-xml_parser_free($xp);
 $xp = xml_parser_create();
 xml_parser_set_option($xp, XML_OPTION_CASE_FOLDING, true);
 xml_set_element_handler($xp, "start_element", "end_element");
 $fp = fopen("xmltest.xml", "r");
 while ($data = fread($fp, 4096)) {
     xml_parse($xp, $data, feof($fp));
 }
-xml_parser_free($xp);
 
 function start_element($xp, $elem, $attribs)
 {
diff --git a/ext/xml/tests/xml009.phpt b/ext/xml/tests/xml009.phpt
@@ -28,7 +28,6 @@ $parser = xml_parser_create_ns("ISO-8859-1","@");
 xml_set_element_handler($parser,'start_elem','end_elem');
 xml_parser_set_option($parser,XML_OPTION_CASE_FOLDING,0);
 xml_parse($parser, $xml);
-xml_parser_free($parser);
 ?>
 --EXPECT--
 string(24) "http://example.com/foo@a"
diff --git a/ext/xml/tests/xml010.phpt b/ext/xml/tests/xml010.phpt
@@ -32,7 +32,6 @@ $parser = xml_parser_create_ns("ISO-8859-1","@");
 xml_set_element_handler($parser,'start_elem','end_elem');
 xml_parser_set_option($parser,XML_OPTION_CASE_FOLDING,0);
 xml_parse($parser, $xml);
-xml_parser_free($parser);
 ?>
 --EXPECT--
 http://example.com/foo@a 
diff --git a/ext/xml/tests/xml011.phpt b/ext/xml/tests/xml011.phpt
@@ -17,7 +17,6 @@ $xml = '<text>start<b /> This &amp; that</text>';
 $parser = xml_parser_create();
 xml_parse_into_struct($parser, $xml, $vals, $index);
 print_r($vals);
-xml_parser_free($parser);
 
 echo "\nChange to empty end handler\n";
 $parser = xml_parser_create();
@@ -26,7 +25,6 @@ xml_set_element_handler($parser,'start_elem','end_elem');
 xml_set_element_handler($parser,'start_elem',NULL);
 xml_parse($parser, $xml, TRUE);
 
-xml_parser_free($parser);
 echo "\nDone\n";
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/xml_closures_001.phpt b/ext/xml/tests/xml_closures_001.phpt
@@ -29,7 +29,6 @@ $fp = fopen("xmltest.xml", "r");
 while ($data = fread($fp, 4096)) {
     xml_parse($xp, $data, feof($fp));
 }
-xml_parser_free($xp);
 
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/xml_error_string_basic.inc b/ext/xml/tests/xml_error_string_basic.inc
@@ -13,5 +13,4 @@ foreach ($xmls as $xml) {
         var_dump(xml_get_error_code($xml_parser));
         var_dump(xml_error_string(xml_get_error_code($xml_parser)));
     }
-    xml_parser_free($xml_parser);
 }
diff --git a/ext/xml/tests/xml_parse_into_struct_variation.phpt b/ext/xml/tests/xml_parse_into_struct_variation.phpt
@@ -9,7 +9,6 @@ echo "*** Testing xml_parse_into_struct() : variation ***\n";
 $simple = "<main><para><note>simple note</note></para><para><note>simple note</note></para></main>";
 $p = xml_parser_create();
 xml_parse_into_struct($p, $simple, $vals, $index);
-xml_parser_free($p);
 echo "Index array\n";
 print_r($index);
 echo "\nVals array\n";
diff --git a/ext/xml/tests/xml_parser_free_deprecated.phpt b/ext/xml/tests/xml_parser_free_deprecated.phpt
@@ -0,0 +1,15 @@
+--TEST--
+xml_parser_free() deprecation message
+--EXTENSIONS--
+xml
+--FILE--
+<?php
+
+$xp = xml_parser_create();
+xml_parse($xp, '<root></root>',TRUE);
+xml_parser_free($xp);
+echo "Done\n";
+?>
+--EXPECTF--
+Deprecated: Function xml_parser_free() is deprecated since 8.5, as it has no effect since PHP 8.0 in %s on line %d
+Done
diff --git a/ext/xml/tests/xml_parser_set_option_basic.phpt b/ext/xml/tests/xml_parser_set_option_basic.phpt
@@ -26,8 +26,6 @@ var_dump(xml_parser_get_option($parser, XML_OPTION_TARGET_ENCODING));
 var_dump(xml_parser_set_option($parser, XML_OPTION_TARGET_ENCODING, "US-ASCII"));
 var_dump(xml_parser_get_option($parser, XML_OPTION_TARGET_ENCODING));
 
-xml_parser_free( $parser );
-
 echo "Done\n";
 ?>
 --EXPECT--
diff --git a/ext/xml/tests/xml_set_notation_decl_handler_basic.phpt b/ext/xml/tests/xml_set_notation_decl_handler_basic.phpt
@@ -32,7 +32,6 @@ class XML_Parser
         xml_set_notation_decl_handler($parser, $this->notation_decl_handler(...));
         xml_set_unparsed_entity_decl_handler($parser, $this->unparsed_entity_decl_handler(...));
         xml_parse($parser, $data, true);
-        xml_parser_free($parser);
     }
 }
 
diff --git a/ext/xml/tests/xml_set_processing_instruction_handler_basic.phpt b/ext/xml/tests/xml_set_processing_instruction_handler_basic.phpt
@@ -18,7 +18,6 @@ class XML_Parser
         $parser = xml_parser_create();
         xml_set_processing_instruction_handler($parser, $this->PIHandler(...));
         xml_parse($parser, $data, true);
-        xml_parser_free($parser);
     }
 
 
diff --git a/ext/xml/tests/xml_set_start_namespace_decl_handler_basic.inc b/ext/xml/tests/xml_set_start_namespace_decl_handler_basic.inc
@@ -14,7 +14,6 @@ var_dump(xml_set_start_namespace_decl_handler( $parser, "Namespace_Start_Handler
 var_dump(xml_set_end_namespace_decl_handler( $parser, "Namespace_End_Handler" ));
 
 xml_parse( $parser, $xml, true);
-xml_parser_free( $parser );
 
 echo "Done\n";
 
diff --git a/ext/xml/xml.stub.php b/ext/xml/xml.stub.php
@@ -189,6 +189,7 @@ function xml_get_current_column_number(XMLParser $parser): int {}
 
 function xml_get_current_byte_index(XMLParser $parser): int {}
 
+#[\Deprecated(since: '8.5', message: "as it has no effect since PHP 8.0")]
 function xml_parser_free(XMLParser $parser): bool {}
 
 /** @param string|int|bool $value */
diff --git a/ext/xml/xml_arginfo.h b/ext/xml/xml_arginfo.h

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,6 @@ function boom()`
`11`	`11`	`$thing = new Thing();`
`12`	`12`	`xml_set_object($reader, $thing);`
`13`	`13`	`die("ok\n");`
`14`		`- xml_parser_free($reader);`
`15`	`14`	`}`
`16`	`15`	`boom();`
`17`	`16`	`?>`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,6 @@ function parseXML2($xml) {`
`30`	`30`	`function parseXML3($xml) {`
`31`	`31`	`$p = xml_parser_create();`
`32`	`32`	`xml_parse_into_struct($p, $xml, $vals, $index);`
`33`		`- xml_parser_free($p);`
`34`	`33`	`return var_export($vals, true);`
`35`	`34`	`}`
`36`	`35`
Original file line number	Diff line number	Diff line change
`@@ -69,5 +69,4 @@ foreach ($xmls as $desc => $xml) {`
`69`	`69`	`xml_set_character_data_handler($xml_parser, "characterData");`
`70`	`70`	`if (!xml_parse($xml_parser, $xml, true))`
`71`	`71`	`echo "Error: ".xml_error_string(xml_get_error_code($xml_parser))."\n";`
`72`		`- xml_parser_free($xml_parser);`
`73`	`72`	`}`