ICU-400.42.tar.gz

[apple/icu.git] / icuSources / layout / LESwaps.h

diff --git a/icuSources/layout/LESwaps.h b/icuSources/layout/LESwaps.h
index 87a07d8dc0eea3a978d626f3f3afac43b596a6ef..5309139d6732e33fa4f7d5b7feb2bed449f8a46b 100644 (file)
--- a/icuSources/layout/LESwaps.h
+++ b/icuSources/layout/LESwaps.h
@@ -1,8 +1,7 @@
 
 /*
- * @(#)LESwaps.h       1.3 00/03/15
  *
- * (C) Copyright IBM Corp. 1998-2003 - All Rights Reserved
+ * (C) Copyright IBM Corp. 1998-2008 - All Rights Reserved
  *
  */
 
@@ -11,57 +10,41 @@
 
 #include "LETypes.h"
 
+/**
+ * \file 
+ * \brief C++ API: Endian independent access to data for LayoutEngine
+ */
+
 U_NAMESPACE_BEGIN
 
 /**
  * A convenience macro which invokes the swapWord member function
  * from a concise call.
  *
- * @draft ICU 2.2
+ * @stable ICU 2.8
  */
-#define SWAPW(value) (LESwaps::isBigEndian() ? (value) : LESwaps::swapWord(value))
-
+#define SWAPW(value) LESwaps::swapWord((const le_uint16 &) (value))
 
 /**
  * A convenience macro which invokes the swapLong member function
  * from a concise call.
  *
- * @draft ICU 2.2
+ * @stable ICU 2.8
  */
-#define SWAPL(value) (LESwaps::isBigEndian() ? (value) : LESwaps::swapLong(value))
+#define SWAPL(value) LESwaps::swapLong((const le_uint32 &) (value))
 
 /**
  * This class is used to access data which stored in big endian order
- * regardless of the conventions of the platform. It has been designed
- * to automatically detect the endian-ness of the platform, so that a
- * compilation flag is not needed.
+ * regardless of the conventions of the platform.
  *
  * All methods are static and inline in an attempt to induce the compiler
  * to do most of the calculations at compile time.
  *
- * @draft ICU 2.2
+ * @stable ICU 2.8
  */
 class U_LAYOUT_API LESwaps /* not : public UObject because all methods are static */ {
 public:
 
-    /**
-     * This method detects the endian-ness of the platform by
-     * casting a pointer to a word to a pointer to a byte. On
-     * big endian platforms the FF will be in the byte with the
-     * lowest address. On little endian platforms, the FF will
-     * be in the byte with the highest address.
-     *
-     * @return true if the platform is big endian
-     *
-     * @draft ICU 2.2
-     */
-    static le_bool isBigEndian()
-    {
-        const le_uint16 word = 0xFF00;
-
-        return *((le_uint8 *) &word);
-    };
-
     /**
      * This method does the byte swap required on little endian platforms
      * to correctly access a (16-bit) word.
@@ -70,11 +53,13 @@ public:
      *
      * @return the byte swapped word
      *
-     * @draft ICU 2.2
+     * @stable ICU 2.8
      */
-    static le_uint16 swapWord(le_uint16 value)
+    static le_uint16 swapWord(const le_uint16 &value)
     {
-        return (((le_uint8) (value >> 8)) | (value << 8));
+        const le_uint8 *p = (const le_uint8 *) &value;
+
+        return ((p[0] << 8) + p[1]);
     };
 
     /**
@@ -85,11 +70,13 @@ public:
      *
      * @return the byte swapped long
      *
-     * @draft ICU 2.2
+     * @stable ICU 2.8
      */
-    static le_uint32 swapLong(le_uint32 value)
+    static le_uint32 swapLong(const le_uint32 &value)
     {
-        return swapWord((le_uint16) (value >> 16)) | (swapWord((le_uint16) value) << 16);
+        const le_uint8 *p = (const le_uint8 *) &value;
+
+        return ((p[0] << 24) + (p[1] << 16) + (p[2] << 8) + p[3]);
     };
 
 private: