[Tcl-bugs] [ tcl-Feature Requests-3598875 ] new macro TclFreeIfRefCountZero

[SourceForge.net <no...@so...>]

Feature Requests item #3598875, was opened at 2012-12-29 10:21
Message generated for change (Comment added) made by ferrieux
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=360894&aid=3598875&group_id=10894

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: 10. Objects
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Jan Nijtmans (nijtmans)
Assigned to: Alexandre Ferrieux (ferrieux)
Summary: new macro TclFreeIfRefCountZero

Initial Comment:
In various places, objects with refCount zero are not
cleaned up in the error-case.

See branch "novem-freeifrefcountzero" for a start,
there are probably many more places where this
macro is useful. Meant for Tcl 9. Ongoing.

Inspired by the discussion regarding bug 3598580
with Alexandre Ferrieux.

----------------------------------------------------------------------

>Comment By: Alexandre Ferrieux (ferrieux)
Date: 2012-12-30 03:36

Message:
Yes, I've been told about this already (and probably by you Donal ;).
Would it be possible to be more precise and factual ?
Please give, in this ticket's comment thread, concrete examples of the
complex/slow idioms that it would/did entail.

----------------------------------------------------------------------

Comment By: Donal K. Fellows (dkf)
Date: 2012-12-30 03:29

Message:
The objects-start-with-refcount-1 approach was tried during the 8.0 alphas,
and we switched to the current system as it greatly simplified most basic
uses. (OK, it's in pre-history as it is from before the switch to CVS, but
I wrote code against 8.0a1 as I desperately needed the performance boost,
and I *remember* what it was like. You had to have a lot of temporary
variables where now you can just feed objects directly into APIs like
Tcl_ListObjAppendObj.)

----------------------------------------------------------------------

Comment By: Jan Nijtmans (nijtmans)
Date: 2012-12-29 10:39

Message:
>"NewObj()_>refCount==1" for novem ?
No, I don't think that's a good idea. (but feel free to prove me wrong in a
separate branch)

----------------------------------------------------------------------

Comment By: Alexandre Ferrieux (ferrieux)
Date: 2012-12-29 10:29

Message:
Interesting. But while we're at it, what about frankly switching to
"NewObj()_>refCount==1" for novem ?
I admit this will seriously threaten ABI compatibility. But is this still a
concern for novem ?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=360894&aid=3598875&group_id=10894