Re: [Gawk manual] HTML issues

2024-10-14 Thread arnold
Hi. I will be pushing the attached diff, which uses @multitable. The HTML isn't perfect, but it simplifies the document considerably. Thanks, Arnold Patrice Dumas wrote: > On Mon, Oct 14, 2024 at 10:29:30AM +0200, Thérèse Godefroy wrote: > > Hello Arnold, hello all, > > > > Le 14/10/2024 à 0

Re: [Gawk manual] HTML issues

2024-10-14 Thread Patrice Dumas
On Mon, Oct 14, 2024 at 10:29:30AM +0200, Thérèse Godefroy wrote: > Hello Arnold, hello all, > > Le 14/10/2024 à 05:40, arn...@skeeve.com a écrit : > > Hello. > > > > Your patch, unfortunately, is not correct, as it ends up putting > > regular text into @code even for non-HTML outputs, so I canno

Re: C++ hash map usage

2024-10-14 Thread Patrice Dumas
On Mon, Oct 14, 2024 at 09:21:50PM +0100, Gavin Smith wrote: > I may as well post what I've been able to come up with. The new > code is about 150 lines of C, in the file convert/hashmap.c (patch > below). Probably not perfect but hopefully simple enough to be > maintainable. > > If this approac

Re: C++ hash map usage

2024-10-14 Thread Gavin Smith
On Mon, Oct 14, 2024 at 01:25:31PM +0100, Gavin Smith wrote: > These numbers do not seem so high that we need an ultra-optimised > algorithm (developed by C++ boffins) to deal with them - anything faster > than a linear search would probably good enough. > > I have some spare time over the next co

Re: do not remove .inf or .info in @ref manual name argument to construct HTML ref

2024-10-14 Thread Patrice Dumas
On Mon, Oct 14, 2024 at 06:33:21PM +0100, Gavin Smith wrote: > > If we continue removing the extension, it should be mentioned in the > > manual in the HTML Xref specification, I think as an exception that > > other implementations do not need to follow, as it should be mentioned > > and also becau

Re: do not remove .inf or .info in @ref manual name argument to construct HTML ref

2024-10-14 Thread Gavin Smith
On Wed, Jan 10, 2024 at 11:53:24PM +0100, Patrice Dumas wrote: > Hello, > > Right now, a .inf or .info extension in fourth @ref/@xref... argument is > removed to determine the manual base name for the purpose of doing cross > references to manuals in HTML. I guess that it comes from the time when

Re: FreeBSD failures with env for in document translations for master

2024-10-14 Thread Patrice Dumas
On Mon, Oct 14, 2024 at 03:54:32PM +0200, Patrice Dumas wrote: > This is a pain, but I think that the simplest would be having a call to > Texinfo::Translations::translate_string replace translate_string in > tp/Texinfo/XS/main/translations.c, similar to what is done already with > the calls in tp/

Re: FreeBSD failures with env for in document translations for master

2024-10-14 Thread Patrice Dumas
On Mon, Oct 14, 2024 at 12:40:42PM +0100, Gavin Smith wrote: > Hovever, I remember that there was more dependencies on gettext in XS > code after the 7.1 release, so this may not be possible. (I'd have to > go back to old emails to remember the details - it may have been > for the rarely used "obj

C++ hash map usage

2024-10-14 Thread Gavin Smith
On Sun, Oct 06, 2024 at 02:28:02PM +0200, Patrice Dumas wrote: > On Sun, Oct 06, 2024 at 12:09:45PM +0100, Gavin Smith wrote: > > On Sun, Oct 06, 2024 at 09:53:22AM +0200, Patrice Dumas wrote: > > > > How much slower would the linear search actually be? > > > > > > It is much slower (if I recall w

Re: FreeBSD failures with env for in document translations for master

2024-10-14 Thread Gavin Smith
On Sun, Oct 13, 2024 at 04:50:48PM +0200, Patrice Dumas wrote: > On Sun, Oct 13, 2024 at 03:30:14PM +0100, Gavin Smith wrote: > > We have had memory errors with setenv in XS code before. > > Might indeed be related, though the issues were much less severe who > knows what could happen on other pla

Re: [Gawk manual] HTML issues

2024-10-14 Thread Thérèse Godefroy
Hello Arnold, hello all, Le 14/10/2024 à 05:40, arn...@skeeve.com a écrit : Hello. Please send any further HTML issues directly to the Texinfo people (cc'ed). I simply use the currently released version of makeinfo to format the documents. Your patch, unfortunately, is not correct, as it ends