[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[libvirt] Re: [PATCH] portability: don't include <endian.h>



John Levon <john levon Sun COM> wrote:
> On Wed, Dec 17, 2008 at 01:55:59PM +0100, Jim Meyering wrote:
>> Adding the inclusions of <string.h> and <mntent.h> look fine,
>
> They were just moved.
>
>> but rather than testing for endian.h, it seems we can do away with
>> it altogether.  Even the __LITTLE_ENDIAN and __BIG_ENDIAN
>> constants aren't really required.  Rather, all you need is
>
> Fine by me, if you prefer it this way.

Of course.
Avoiding an #ifdef like that is worth an iteration.

I've committed the patch I proposed, with one change:
also remove the unnecessary inclusion of <byteswap.h>.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]