Re: [PATCH 0/6] newlib/Cygwin: Allow more inlining of locale_t stuff

[Corinna Vinschen <vinschen at redhat dot com>]

On Nov 14 08:23, Sebastian Huber wrote:
> On 13/11/17 16:59, Corinna Vinschen wrote:
> > Having said that, I'm not overly happy with the Cygwin __getreent
> > change.  I'm not sure if performance is worth the price here.
> 
> There is also a price in terms of memory space here if we initialize
> _REENT->_locale with __global_locale. Applications not using this
> ctype/locale stuff will now also include the __global_locale.

Isn't that always, or most of the time, the case anyway?  The
ctype functions call __locale_ctype_ptr() which in turn calls
__get_current_locale():

  return _REENT->_locale ?: __get_global_locale ();


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: signature.asc
Description: PGP signature