texinfo-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

branch master updated: * tp/Texinfo/XS/parsetexi/Parsetexi.pm (parser):


From: Patrice Dumas
Subject: branch master updated: * tp/Texinfo/XS/parsetexi/Parsetexi.pm (parser): never set the 'conf' parser key, it is not used since the time parser conf became internal only.
Date: Sat, 19 Oct 2024 17:57:17 -0400

This is an automated email from the git hooks/post-receive script.

pertusus pushed a commit to branch master
in repository texinfo.

The following commit(s) were added to refs/heads/master by this push:
     new 0cd36347c7 * tp/Texinfo/XS/parsetexi/Parsetexi.pm (parser): never set 
the 'conf' parser key, it is not used since the time parser conf became 
internal only.
0cd36347c7 is described below

commit 0cd36347c7f1e74e4afea0c95f8040f5941dd1db
Author: Patrice Dumas <pertusus@free.fr>
AuthorDate: Sat Oct 19 23:57:13 2024 +0200

    * tp/Texinfo/XS/parsetexi/Parsetexi.pm (parser): never set the 'conf'
    parser key, it is not used since the time parser conf became internal
    only.
---
 ChangeLog                            |  6 ++++++
 tp/Texinfo/XS/parsetexi/Parsetexi.pm | 29 +----------------------------
 2 files changed, 7 insertions(+), 28 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 98c18fec2c..79d2496e31 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2024-10-19  Patrice Dumas  <pertusus@free.fr>
+
+       * tp/Texinfo/XS/parsetexi/Parsetexi.pm (parser): never set the 'conf'
+       parser key, it is not used since the time parser conf became internal
+       only.
+
 2024-10-19  Patrice Dumas  <pertusus@free.fr>
 
        * tp/Texinfo/ParserNonXS.pm (parser),
diff --git a/tp/Texinfo/XS/parsetexi/Parsetexi.pm 
b/tp/Texinfo/XS/parsetexi/Parsetexi.pm
index a137dba630..c601751cc1 100644
--- a/tp/Texinfo/XS/parsetexi/Parsetexi.pm
+++ b/tp/Texinfo/XS/parsetexi/Parsetexi.pm
@@ -54,37 +54,14 @@ sub parser (;$)
 {
   my $conf = shift;
 
-  # In Texinfo::Common because all the
-  # customization options information is gathered here, and also
-  # because it is used in other codes, in particular the XS parser.
-  # Note that it also contains inner options like accept_internalvalue
-  # and customizable document parser state values in addition to
-  # regular customization options.
-  my $parser_conf = dclone(\%Texinfo::Common::parser_document_parsing_options);
   my $parser = {};
   bless $parser;
 
-  # Reset conf from argument, restricting to parser_document_parsing_options
-  if (defined($conf)) {
-    foreach my $key (keys(%$conf)) {
-      if (exists($Texinfo::Common::parser_document_parsing_options{$key})) {
-        if (ref($conf->{$key})) {
-          $parser_conf->{$key} = dclone($conf->{$key});
-        } else {
-          $parser_conf->{$key} = $conf->{$key};
-        }
-      } else {
-        # no warning here as in pure Perl as it is warned below
-        #warn "ignoring parser configuration value \"$key\"\n";
-      }
-    }
-  }
-
   # pass directly DEBUG value to reset_parser to override previous
   # parser configuration, as the configuration isn't already reset and the new
   # configuration is set afterwards.
   my $debug = 0;
-  $debug = $parser_conf->{'DEBUG'} if ($parser_conf->{'DEBUG'});
+  $debug = $conf->{'DEBUG'} if ($conf->{'DEBUG'});
 
   # The reset_parser call resets the conf to the same values as found in
   # Texinfo::Common parser_document_parsing_options.
@@ -157,10 +134,6 @@ sub parser (;$)
   }
   if ($store_conf) {
     register_parser_conf($parser);
-
-    # variables set to the parser initialization values
-    # only.  What is found in the document has no effect.
-    $parser->{'conf'} = $parser_conf;
   }
 
   $parser->{'registrar'} = Texinfo::Report::new();



reply via email to

[Prev in Thread] Current Thread [Next in Thread]