On Sun, Oct 06, 2024 at 12:47:42PM +0100, Gavin Smith wrote:
> 
> I've tried it myself although have been unable to run it on anything
> but texinfo.texi for the manuals I tried.
> 
> I did not see anything in teximakehtml.c about which converter was used
> so I patched the code in converter_converter in converter/converter.c
> which passed the CONVF_cxx_hashmap or CONVF_string_list argument to
> new_converter.  (Is the "string list" implementation really used by
> default by texi2any as it doesn't look that way from the code?)

No, the CONVF_cxx_hashmap is used by default if found.

>   /* NOTE if HAVE_CXX_HASHMAP is not set, even with CONVF_cxx_hashmap
>      string lists will be used */
>   size_t converter_descriptor = new_converter (format, CONVF_cxx_hashmap);
>    /*
>    To use a string list.  Slower.
>   size_t converter_descriptor = new_converter (format, CONVF_string_list);
>     */

Note that I have made it simpler to pass flags down to the converter in
this recent commit:
https://git.savannah.gnu.org/cgit/texinfo.git/commit/?id=9b6d9f8f81cb7c7308063108f2df7607eb8b9996

> Times are in seconds.
> 
> manual                C++ hash                string list
> texinfo               0.39                    0.45
> 
> libc.texi - unable to run test due to numerous error messages that
> don't occur with texi2any:

The @iftex sections are expanded, that's why there are all those errors.
They probably do not mess that much the timing, but still this should
not happen.  Strangely I have not seen that when comparing texi2any and 
teximakehtml output, but it may have changed.  I'll fix that.

-- 
Pat

Reply via email to