This is the mail archive of the mailing list for the glibc 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] Use __unused0 instead of __unused for user visible struct members

On Mon, Oct 28, 2013 at 12:18:35PM +0000, Justin Cormack wrote:
> A patch was submitted for this a while back
> and did not
> get a good reception from the maintainer at the time. Attached is an
> updated version for current glibc head.
> The issue here is that use of __unused as a struct member name
> conflicts with the historic use of __unused on BSD systems and code
> for __attribute__((__unused__)) - unfortunately this has become part
> of the user visible API of these systems. In my case I am trying to
> cross compile NetBSD on ppc Linux and there is no way to do this
> without patching the headers. In general most uses of __unused are
> already __unused1, __unused2 etc, it is just a few cases that are not.
> These fields are not part of the user API so these changes should not
> affect any code outside glibc itself.

While it doesn't mean I disagree with this change (because such apps
are broken nonsense and need to be fixed) I wouldn't be surprised if
it breaks at least a few apps which are accessing __unused members by
name (e.g. to zero-fill padding).


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