On Sun, Jan 28, 2024 at 09:25:38PM +0100, Patrice Dumas wrote: > On Sun, Jan 28, 2024 at 08:15:33PM +0000, Gavin Smith wrote: > > Below is a patch to introduce a new variable to avoid using the > > Unicode::Collate module. Turning the module off shortens run times by > > about 5% (depending on the size of indices in the document). > > > > Users can use this if they don't find texi2any fast enough, if they > > don't care about having the indices sorted correctly, if they don't > > have many non-ASCII characters in index entry text, or just for working > > on a manual. > > > > I propose that USE_UNICODE_COLLATE is on by default, as is currently > > the case, to have correct index sorting by default, as the impact is > > relatively small. > > With not unicode collation, the index sorting could be done in C, and my > guess is that the impact would be important.
It might be - would I be right in saying that this code isn't written? > > Patrice, is this the correct way to add a customization variable (as > > I am not familiar with the new Texinfo/options_data.txt file). > > Yes, it looks good. As a side note, shouldn't the variable name rather > be USE_UNICODE_COLLATION? I was referring to the Unicode::Collate module but USE_UNICODE_COLLATION is just as good a name if not better (as it is grammatical English and not referring so much to the implementation details of which module is used to accomplish it).
