On Sun, Feb 21, 2021 at 12:40:40AM +0100, Patrice Dumas wrote:
> Hello,
> 
> Just tried
> 
> ./texi2any.pl -c CHECK_NORMAL_MENU_STRUCTURE=1 test_b.texi
> 
> And I get a warning
>  ignoring parser configuration value "CHECK_NORMAL_MENU_STRUCTURE"
> 
> It is from
> tp/Texinfo/XS/parsetexi/Parsetexi.pm
> 
> I think that it is because it is in Common.pm %default_customization_values
> such that it is passed to the parser in texi2any.pl l 1078, but
> it is not whitelisted in tp/Texinfo/XS/parsetexi/Parsetexi.pm.  I see
> that l 1260 of texi2any.pl Texinfo::Structuring::nodes_tree is indeed
> called with a parser object.
> 
> What about splitting %default_customization_values in two,
>  %default_parser_customization_values
> and
>  %default_structure_customization_values
> and put the options that are not rlevant for the parser in 
> %default_structure_customization_values
> and modify tp/Texinfo/XS/parsetexi/Parsetexi.pm such that it does not
> warn about keys in %default_structure_customization_values?

That sounds sensible.   It would make it easier to check that the
customization variables for the parser were being handled.
Maybe only the variables in %default_structure_customization_values
could be set on the parser object.

Reply via email to