Re: [TCLCORE] CFV warning: TIP #717: New function: Tcl_AttemptCreateHashEntry()

[Ashok N. <apn...@ya...>]

Just for the record, I would still prefer a stub entry even if the panic gives more useful information with the macro.

/Ashok
________________________________
From: Jan Nijtmans <jan...@gm...>
Sent: Tuesday, May 6, 2025 1:20 AM
To: Ashok Nadkarni <apn...@ya...>
Subject: Re: [TCLCORE] CFV warning: TIP #717: New function: Tcl_AttemptCreateHashEntry()

Op ma 5 mei 2025 om 21:39 schreef Jan Nijtmans:
> > I do not understand why the implementation does not simply define a new stub entry for Tcl_AttemptCreateHashEntry instead of introducing all that macro bloat.

Reading this question again, you were talking about 'stub entry', It
would indeed be
possible to create a new stub entry for that. Doing it inline gives
the possibility
for the Tcl_Panic to provide much more useful information. If TCL_MEM_DEBUG
is defined, this is fully utilized: If the Tcl_CreateHashEntry() call fails,
the panic will give the filename and linenumber where it happened.

Hope this helps,
      Jan Nijtmans