This is the mail archive of the mailing list for the newlib project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] New option --disable-reenterable-atexit

On 2013-10-16 12:18, Ye Joey wrote:
On Wed, Oct 16, 2013 at 4:34 PM, Sebastian Huber
<> wrote:
+     Disable reenterablility of atexit by declare related data structures
+     as global variables. By doing these data structures can be discarded
+     if atexit is not referenced.
+     Enabled by default.

I think the naming and the description is a bit misleading.  The atexit()
support is not thread specific.  Operations are serialized with the
__atexit_lock lock.  In case _REENT_GLOBAL_ATEXIT is undefined, then the
data structures for the atexit() support are part of struct _reent, but only
used in _GLOBAL_REENT.  Thus the space in the thread specific structures is
wasted. See also
I named it to mean that atexit isn't in _reent. But _reent isn't a
good name as option as it is implementation specific data structure

How about --enable-global-atexit-data, and changes the description accordingly?

Yes, I think this name describes it better.

Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  :
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]