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

C Howland <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <CANk6obSQ8R1GgXRD_2ehKmhfUsoEbYXYJj2xBHXtbo-UthT4jQ@mail.gmail.com>
>
>
> ------------------------------
> *From:* Newlib <[email protected]> on
> behalf of Sebastian Huber <[email protected]>
> *Sent:* Tuesday, June 21, 2022 8:49 AM
> *To:* [email protected] <[email protected]>
> *Subject:* [PATCH 03/14] Add _REENT_ERRNO(ptr)
>
>
>
> From: Matt Joyce <[email protected]>
>
> Add a _REENT_ERRNO() macro to encapsulate the access to the
> _errno member of struct reent. This will help to replace the
> structure member with a thread-local storage object in a follow
> up patch.
> ---
>

There already exists an __errno_r() macro that does the very same function
(defined in sys/errno.h).  (Its use, however, is limited, only being used
in files under iconv/lib.)  Having the same thing done both ways probably
doesn't make sense.  The new name is more consistent with the rest of the
things being done, while the old name is established and errno is a more
specialized case.  It probably would be a good idea to either
1)  use __errno_r() instead of creating _REENT_ERRNO() or
2)  replace __errno_r() with _REENT_ERRNO() as part of adding the latter.
Craig
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.