X-Recipient: archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status: No, hits=-0.5 required=5.0	tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,T_TO_NO_BRKTS_FREEMAIL
X-Spam-Check-By: sourceware.org
MIME-Version: 1.0
In-Reply-To: <4C647D32.1090907@redhat.com>
References: <AANLkTimQfeALLK=x_UvJYLjusd6ygCtjUKDrdA91e7Ej AT mail DOT gmail DOT com>	<20100812225838 DOT GB15724 AT ednor DOT casa DOT cgf DOT cx>	<4C647D32 DOT 1090907 AT redhat DOT com>
Date: Thu, 12 Aug 2010 21:12:46 -0300
Message-ID: <AANLkTikCc0gdoVTWfhp=xoNY_m=Up4XQiApLmm9K9da+@mail.gmail.com>
Subject: Re: diff /usr/include/endian.orig.h /usr/include/endian.h > endian.h.diff
From: Pedro Izecksohn <pedro DOT izecksohn AT gmail DOT com>
To: cygwin AT cygwin DOT com
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-IsSubscribed: yes
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
Precedence: bulk
List-Id: <cygwin.cygwin.com>
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie DOT com AT cygwin DOT com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com

--- Eric Blake wrote:
> Christopher Faylor wrote:
>> I wrote:
>>> =C2=A0I hope this list accepts attachments.
>>
>> It does but the list mind-reading gizmo is on the fritz.
>
> Translation - a ChangeLog entry justifying your changes, a diff in
> unified or context format (-u or -c) rather than the default ed format,
> and attaching the patch inline with a text/plain MIME type instead of
> base64 encoded, are all necessary before anyone is likely to act on your
> proposed patch.

Defines macros for to convert the endianness of 16, 32 and 64 bits
integer types.

diff -c /usr/include/endian.orig.h /usr/include/endian.h
*** /usr/include/endian.orig.h	Mon Apr 12 14:09:58 2010
--- /usr/include/endian.h	Thu Aug 12 19:40:20 2010
***************
*** 36,40 ****
  #elif __BYTE_ORDER =3D=3D __BIG_ENDIAN
  # define __LONG_LONG_PAIR(HI, LO) HI, LO
  #endif
- #endif /*_ENDIAN_H_*/

--- 36,82 ----
  #elif __BYTE_ORDER =3D=3D __BIG_ENDIAN
  # define __LONG_LONG_PAIR(HI, LO) HI, LO
  #endif

+ #ifdef _BSD_SOURCE
+ #if __BYTE_ORDER =3D=3D __LITTLE_ENDIAN
+
+ #include <byteswap.h>
+
+ #define htobe16(x) bswap_16(x)
+ #define htobe32(x) bswap_32(x)
+ #define htobe64(x) bswap_64(x)
+
+ #define be16toh(x) bswap_16(x)
+ #define be32toh(x) bswap_32(x)
+ #define be64toh(x) bswap_64(x)
+
+ #define htole16(x) (x)
+ #define htole32(x) (x)
+ #define htole64(x) (x)
+
+ #define le16toh(x) (x)
+ #define le32toh(x) (x)
+ #define le64toh(x) (x)
+
+ #else /* __BYTE_ORDER =3D=3D __BIG_ENDIAN */
+
+ #define htobe16(x) (x)
+ #define htobe32(x) (x)
+ #define htobe64(x) (x)
+
+ #define be16toh(x) (x)
+ #define be32toh(x) (x)
+ #define be64toh(x) (x)
+
+ #define htole16(x) bswap_16(x)
+ #define htole32(x) bswap_32(x)
+ #define htole64(x) bswap_64(x)
+
+ #define le16toh(x) bswap_16(x)
+ #define le32toh(x) bswap_32(x)
+ #define le64toh(x) bswap_64(x)
+
+ #endif
+ #endif /*_BSD_SOURCE*/
+
+ #endif /*_ENDIAN_H_*/

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple