This fixes a number of things that are warnings in the Mac compiler....

import/cross/memory_stream.h

 /*
  * Copyright 2009, Google Inc.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 37 matching lines @@
 
 #include <string.h>
 #include "base/basictypes.h"
 
 namespace o3d {
 
 // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 class MemoryReadStream {
  public:
   MemoryReadStream(const uint8 *memory, size_t n)
-      : memory_(memory), length_(n), read_index_(0) {}
+      : memory_(memory), read_index_(0), length_(n) {}
 
   // Explicit copy constructor
   MemoryReadStream(const MemoryReadStream &stream)
       : memory_(stream.memory_),
-        length_(stream.length_),
-        read_index_(stream.read_index_) {}
+        read_index_(stream.read_index_),
+        length_(stream.length_) {}
 
   virtual ~MemoryReadStream() {}
 
   // Similar to fread(), will try to copy |n| bytes to address |p|
   // (copies fewer bytes if the stream doesn't have enough data)
   // returns the number of bytes copied
   virtual size_t Read(void *p, size_t n) {
     size_t remaining = GetRemainingByteCount();
     size_t bytes_to_read = (n < remaining) ? n : remaining;
     memcpy(p, memory_ + read_index_, bytes_to_read);
@@ skipping 97 matching lines @@
   void operator=(const MemoryReadStream&);
 };
 
 // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 class MemoryWriteStream {
  public:
   // May be completely initialized by calling Assign()
   MemoryWriteStream() : memory_(NULL), write_index_(0), length_(0) {}
 
   MemoryWriteStream(uint8 *memory, size_t n)
-      : memory_(memory), length_(n), write_index_(0) {}
+      : memory_(memory), write_index_(0), length_(n) {}
 
   // Explicit copy constructor
   MemoryWriteStream(const MemoryWriteStream &stream)
       : memory_(stream.memory_),
-        length_(stream.length_),
-        write_index_(stream.write_index_) {}
+        write_index_(stream.write_index_),
+        length_(stream.length_) {}
 
   virtual ~MemoryWriteStream() {}
 
   // In the case where the default constructor was used, this is useful
   // to assign a pointer to memory and a length in bytes.
   void Assign(uint8 *memory, size_t n) {
     memory_ = memory;
     length_ = n;
     write_index_ = 0;
   }
@@ skipping 68 matching lines @@
 class StreamProcessor {
  public:
   virtual ~StreamProcessor() {}
   virtual int ProcessBytes(MemoryReadStream *stream,
                            size_t bytes_to_process) = 0;
 };
 
 }  // namespace o3d
 
 #endif  // O3D_IMPORT_CROSS_MEMORY_STREAM_H_