On Fri, Oct 16, 2026 at 11:07:22AM +0100, Gavin Smith wrote: > > I can reproduce it with the development sources (line numbers changed > > widely, though). Note that it happens only with pure Perl, with > > TEXINFO_XS=omit. > > I found the same. > > I noticed that "TEXINFO_XS=omit texi2any --paragraphindent=asis --info a.texi" > still works correctly. We have special processing of the --paragraphindent > option to convert the argument to the internal value for the paragraphindent > variable. > > I don't think it is necessary to make setting 'paragraphindent' to a > keyword argument work as it is a numeric variable internally and the > --paragraphindent option exists. So one option is just to document the > special values, in case somebody uses this variable: 0 for 'none', -2 for > asis.
I think that it is right to do that, as it would be better in the future to avoid mixed-typed variables. So starting with some documentation now is good. > Otherwise, it could be slightly complicated to add special handling for > this everywhere the user may set customisation variables: from the > command-line, but also from Perl customisation files. In > perl/Texinfo/Config.pm, GNUT_set_from_cmdline and texinfo_set_from_init_file > both look relevant (there would also be the XS/C versions, as well). I don't think that it would be so difficult, and since init files are read only at the beginning it would only be needed to do it in the converter initialization, so no issue of efficiency. What I propose is to do the translation from asis/none to numerical values, but warn. -- Pat
