Rename struct ucontext tag (bug 21457)

Joseph Myers joseph@codesourcery.com
Thu Jun 15 22:02:00 GMT 2017


On Tue, 13 Jun 2017, Florian Weimer wrote:

> > In this case, the type does need to have some tag rather than just a
> > typedef name, because it includes a pointer to itself.  This patch
> > uses struct ucontext_t as the new tag, so the type is mangled as
> > ucontext_t (the POSIX *_t reservation applies in all namespaces, not
> > just the namespace of ordinary identifiers).  Another reserved name
> > such as struct __ucontext could of course be used.
> 
> How widely known is the reserved nature of _t names?  Maybe the __
> prefix would communicate better that this is an internal name?  Although
> I assume we will never change it again, so applications referencing the
> new name should not face any future problems.

Since this is a (C++) ABI, we don't want to change it unnecessarily, so I 
don't see a reason for subsequent changes.  We just need to pick the name 
we want to use (and then people with patches such as the RISC-V port that 
add sys/ucontext.h files will need to follow accordingly).

-- 
Joseph S. Myers
joseph@codesourcery.com



More information about the Libc-alpha mailing list