Fw: [PATCH 03/14] Add _REENT_ERRNO(ptr)

Corinna Vinschen vinschen@redhat.com
Wed Jul 13 11:13:22 GMT 2022


On Jul 13 10:19, Sebastian Huber wrote:
> On 13/07/2022 09:50, Corinna Vinschen wrote:
> > On Jul 13 09:17, Sebastian Huber wrote:
> > > On 12/07/2022 20:25, Corinna Vinschen wrote:
> > > > > What about renaming the existing __errno_r() uses in Newlib to
> > > > > _REENT_ERRNO() with a definition of __errno_r() in <sys/errno.h> for
> > > > > potential users outside of Newlib?
> > > > Sounds good to me.
> > > Attached is a new version of this patch. In this patch the uses of
> > > __errno_r() were replaced with _REENT_ERRNO().
> > In future, would you mind to send followup patches as v2, v3, the same
> > way as the first patch, please?  It's not required as such, but it makes
> > applying a series easier.
> 
> Ok, sorry.
> 
> > 
> > Having said that, the patchset looks good to me.  AFAICS this also
> > uncovered a minor bug in Cygwin in the errno handling, but that's
> > something for us Cygwin folks to patch after you applied your set.
> 
> Thanks for the review, I checked it in.

Thanks.  To close this, the alleged bug in Cygwin's errno handling isn't
a bug, just a misunderstanding on my side.


Corinna



More information about the Newlib mailing list