Re: [Xen-devel] [PATCH 2/5] xen: Add headers to include/Makefile

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 31.05.12 at 17:07, Jean Guyader <jean.guyader@xxxxxxxxxx> wrote:

>   Add stdlib.h for size_t
>   Add string.h for memcpy
>   Add sys/types.h for ssize_t
> 
> Signed-off-by: Jean Guyader <jean.guyader@xxxxxxxxxx>

Sorry, I don't think this is correct: What you're modifying is the
rule to validate public headers, and public headers should never
include any of the three above - the only headers consuming
defined symbols of which is half-way acceptable are those that
exclusively deal with compiler properties (stddef.h, stdint.h,
stdbool.h, stdarg.h, and maybe inttypes.h).

Plus I don't see how public headers would legitimately and
portably (keep 32-on-64 in mind!) use any of the three symbols
mentioned above.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel