Re: [PATCH v2 1/7] banned-die: create header for banning of functions
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Aug 25, 2026, 20:34 UTC
- Message-ID
- <xmqqh5kikkgi.fsf@gitster.g>
- In-Reply-To
- <84634717e2eca479026d1cdf39a089a8f61d131e.1787684181.git.gitgitgadget@gmail.com>
"Derrick Stolee via GitGitGadget" <gitgitgadget@gmail.com> writes:
Show 17 quoted lines
> From: Derrick Stolee <stolee@gmail.com> > > We have universally-banned functions listed in banned.h since > c8af66ab8ad (automatically ban strcpy(), 2018-07-26), but some layers of > the code should be more strict than others. > > One such example is the trace2 API which runs during atexit() and can > prove to cause die()-handler recursion problems if it calls die(). > > Create a new banned-die.h header file that will ban some Git methods > that call die(). Include that in all trace2 API implementation files. > This currently only bans die() itself, and that was already not used. > > It would be reasonable to name this file trace2/tr2_banned.h to be > specific to the trace2 API, but it seems like such a restriction would > be valuable to put in some other areas of the code, so adding it at the > root of the tree seems like a good long-term approach.
In other words, the functions banned by including this file are not listed because they are banned from being used in trace2 API, but because they may lead to die(). There may be some other traits that we might want to avoid in certain subset of our code, and we may have similar banned-frotz.h header to prevent direct or indirect use of frotz. Which makes sense to me.
Would the same approach work for the_hash_algo and the_repository, I wonder?