Subject: bug#79035: Remove micro-optimization for Funintern for performance

From: Eli Zaretskii <[email-masked]>
Date: Thu, 17 Jul 2025 09:44:22 +0300

> From: Lynn Winebarger <[email-masked]>
> Date: Thu, 17 Jul 2025 01:24:03 -0400
> Cc: [email-masked], Mattias Engdegård <[email-masked]>, 
> 	Stefan Monnier <[email-masked]>
> 
>  > The total reported cpu time for
>  > a8b65860a5e28ee0867e8506a17d74d4a9b7783a is 3479.51s, and with the
>  > patch it is 3460.69.
> 
>  That's just 0.5% of speedup, which doesn't pass my threshold for
>  worthy changes, especially when the code changes are non-trivial (as
>  they are in this case).  So my vote is against installing this, sorry.
> 
> The performance improvement is the punchline.  The purpose of the change is to remove a static variable. 
> That's why I originally included it in the patch to make the code in lread.c re-entrant.  But it  probably makes
> more sense as a stand-alone change.

If this change is related to some other changeset, let alone is needed
by it, it is not standalone, and should be part of that other
changeset.  It would make sense to have it as standalone if it brought
us some significant advantages, which doesn't seem to be the case
here.