|
Message-ID: <CALS3df2vc8V4-rv3ThT=sPQTVBbODsf9Xh2T6zLt_5dC2usNxg@mail.gmail.com> Date: Sun, 4 May 2014 04:52:28 +0200 From: Paweł Dziepak <pdziepak@...rnos.org> To: Rich Felker <dalias@...c.org> Cc: musl@...ts.openwall.com Subject: Re: [PATCH] add definition of max_align_t to stddef.h 2014-05-01 0:43 GMT+02:00 Rich Felker <dalias@...c.org>: > On Wed, Apr 30, 2014 at 11:42:51PM +0200, Szabolcs Nagy wrote: >> * Pawel Dziepak <pdziepak@...rnos.org> [2014-04-30 22:23:01 +0200]: >> > >> > +TYPEDEF union { long double ld; long long ll; } max_align_t; >> >> this is wrong >> >> - ld and ll identifiers are not reserved for the implementation >> (you could name them _ld, _ll or __ld, __ll etc) > > It's also not clear to me why this should go in alltypes.h. Unless it > needs to be arch-specific, it could go directly in stddef.h. I thought it would be better if types required by the standard are defined in one file. It doesn't really look like alltypes.h is reserved for arch-specific types. On the other hand, there indeed is not any reason not to put that definition directly in stddef.h. Paweł
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.