fix size_t/int warnings

include/core/SkEndian.h

Index: include/core/SkEndian.h
diff --git a/include/core/SkEndian.h b/include/core/SkEndian.h
index 6eba297548ee0a88d63b2e11e7c0fca4f95d1786..ad0e2a0ca0147cdf4367875baf6e433022bbc83e 100644
--- a/include/core/SkEndian.h
+++ b/include/core/SkEndian.h
@@ -29,7 +29,7 @@
 /** Swap the two bytes in the low 16bits of the parameters.
     e.g. 0x1234 -> 0x3412
 */
-static inline uint16_t SkEndianSwap16(U16CPU value) {
+static inline uint16_t SkEndianSwap16(size_t value) {

[mtklein, 2014/04/23 13:36:18]

If you feel like being a little fancier, I think

template <typename T>
static inline uint16_t SkEndianSwap16(T value) {
   ...
}

conveys the we-take-anything nature of the function.

[reed1, 2014/04/23 14:04:22]

don't want to take any non-integral values (e.g. ptrs, floats, etc). Is size_t
sufficient, or would uintptr_t be cleaner?

     SkASSERT(value == (uint16_t)value);
     return static_cast<uint16_t>((value >> 8) | (value << 8));
 }
@@ -52,11 +52,12 @@ static inline void SkEndianSwap16s(uint16_t array[], int count) {
 /** Reverse all 4 bytes in a 32bit value.
     e.g. 0x12345678 -> 0x78563412
 */
-static inline uint32_t SkEndianSwap32(uint32_t value) {
-    return  ((value & 0xFF) << 24) |
-            ((value & 0xFF00) << 8) |
-            ((value & 0xFF0000) >> 8) |
-            (value >> 24);
+static inline uint32_t SkEndianSwap32(size_t value) {

[mtklein, 2014/04/23 13:36:18]

This guy seems like he really wants to be u32 -> u32.  Maybe the fix is doing
the SkToU32 outside the call?

[reed1, 2014/04/23 14:04:22]

well, so does swap16, but the caller may give us larger types

+    uint32_t v = SkToU32(value);
+    return  ((v & 0xFF) << 24) |
+            ((v & 0xFF00) << 8) |
+            ((v & 0xFF0000) >> 8) |
+             (v >> 24);
 }
 template<uint32_t N> struct SkTEndianSwap32 {
     static const uint32_t value = ((N & 0xFF) << 24) |