Pulled out EndianSwap into its own function.

Change-Id: I4a587102db8c435b739c92b6c464c94c4ea1fe42

include/flatbuffers/flatbuffers.h

@@ -151,10 +151,7 @@ inline void EndianCheck() {
   (void)endiantest;
 }
 
-template<typename T> T EndianScalar(T t) {
-  #if FLATBUFFERS_LITTLEENDIAN
-    return t;
-  #else
+template<typename T> T EndianSwap(T t) {
   #if defined(_MSC_VER)
     #pragma push_macro("__builtin_bswap16")
     #pragma push_macro("__builtin_bswap32")
@@ -163,8 +160,6 @@ template<typename T> T EndianScalar(T t) {
     #define __builtin_bswap32 _byteswap_ulong
     #define __builtin_bswap64 _byteswap_uint64
   #endif
-    // If you're on the few remaining big endian platforms, we make the bold
-    // assumption you're also on gcc/clang, and thus have bswap intrinsics:
   if (sizeof(T) == 1) {   // Compile-time if-then's.
     return t;
   } else if (sizeof(T) == 2) {
@@ -184,6 +179,13 @@ template<typename T> T EndianScalar(T t) {
     #pragma pop_macro("__builtin_bswap32")
     #pragma pop_macro("__builtin_bswap64")
   #endif
+}
+
+template<typename T> T EndianScalar(T t) {
+  #if FLATBUFFERS_LITTLEENDIAN
+    return t;
+  #else
+    return EndianSwap(t);
   #endif
 }