bug#65491: [PATCH] Improve performance allocating vectors

[Mattias Engdegård]

16 sep. 2023 kl. 20.19 skrev Eli Zaretskii <eliz@gnu.org>:

> I get it that you are confident, but I want to be confident as well,
> and I'm not there yet.

That's fine, I'm in no rush. I'll be happy to answer any questions you may have.

It's natural that our perspectives are different. From my point of view I have 
a well-researched and carefully tested change with everything indicating that 
it's safe and an improvement. But when the first thing you see is a flood of 
warnings, it's quite understandable that it is taken as a sign that something 
is wrong.

> Those special configurations have telltale traits that can be used in
> cpp conditionals.  IOW, we could have different definitions of XUNTAG
> for different configurations.  It isn't unheard off, and other macros,
> including some that are involved in XUNTAG, do indeed have separate
> definitions for several configurations.

Certainly, but we didn't need multiple definitions for XUNTAG before and 
nothing has changed in that respect.

However, if you think that it would make the code easier to understand we could 
separate the code into two or more #if clauses, although most of the time a 
single definition is easier to maintain.
In any case, that would be a separate change.