bug-texinfo
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Texinfo 7.0.94 pretest available


From: Gavin Smith
Subject: Re: Texinfo 7.0.94 pretest available
Date: Sun, 15 Oct 2023 12:57:46 +0100

On Sun, Oct 15, 2023 at 12:00:47PM +0300, Eli Zaretskii wrote:
> Thanks.
> 
> This doesn't compile with MinGW, because some of the dTHX additions I
> needed for the previous pretest were not installed(?), and are still
> missing.  The patch I needed is below.

I hadn't installed those changes because I hadn't understood why they
were necessary.  Looking at the code, I expect that it was due to the
use of fprintf in those functions which the Perl headers must be doing
something funny with.  Presumably you got an error message when compiling
indicating this?

The fprintf calls were added since the Texinfo 7.0 branch so will not
have broken previously.

Although adding dTHX should be harmless, the paragraph formatting
functions are very frequently called functions and adding the dTHX in
there has a potential performance impact, especially in xspara__add_next.

However, I could not detect any performance difference in the testing I
did so I have added them anyway.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]