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

import/cross/collada.cc

 /*
  * 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 147 matching lines @@
 // Parameters:
 //   pack:      The pack into which the scene objects will be placed.
 // Returns true on success.
 Collada::Collada(Pack* pack, const Options& options)
     : service_locator_(pack->service_locator()),
       pack_(pack),
       options_(options),
       dummy_effect_(NULL),
       dummy_material_(NULL),
       instance_root_(NULL),
+      collada_zip_archive_(NULL),
       cull_enabled_(false),
       cull_front_(false),
       front_cw_(false),
-      collada_zip_archive_(NULL),
       unique_filename_counter_(0) {
 }
 
 Collada::~Collada() {
   delete collada_zip_archive_;
 }
 
 void Collada::ClearData() {
   textures_.clear();
   original_data_.clear();
@@ skipping 319 matching lines @@
     if (fcd_curve != NULL) {
       FunctionEval* function_eval = pack_->Create<FunctionEval>();
       BindParams(function_eval, FunctionEval::kInputParamName, animation_input);
 
       Curve* curve = pack_->Create<Curve>();
       function_eval->set_function_object(curve);
 
       curve->set_pre_infinity(ConvertInfinity(fcd_curve->GetPreInfinity()));
       curve->set_post_infinity(ConvertInfinity(fcd_curve->GetPostInfinity()));
 
-      for (int i = 0; i != fcd_curve->GetKeyCount(); ++i) {
+      for (unsigned int i = 0; i != fcd_curve->GetKeyCount(); ++i) {
         FCDAnimationKey* fcd_key = fcd_curve->GetKey(i);
         switch (fcd_key->interpolation) {
           case FUDaeInterpolation::STEP:
             BuildStepKey(curve, fcd_key, output_scale);
             break;
           case FUDaeInterpolation::BEZIER:
             BuildBezierKey(curve, static_cast<FCDAnimationKeyBezier*>(fcd_key),
                            output_scale);
             break;
           default:
@@ skipping 983 matching lines @@
                 source_stream.semantic(),
                 source_stream.semantic_index(),
                 dest_fields[ii],
                 0)) {
               O3D_ERROR(service_locator_)
                 << "could not SetVertexStream on StreamBank '"
                 << new_stream_bank->name() << "'";
               return NULL;
             }
           }
+          default:
+            // do nothing
+            break;
         }
       }
       if (!copied) {
         // It's a shared field, copy it to the shared buffer.
         source_fields[ii] = shared_buffer->CreateField(field.GetClass(),
                                                        field.num_components());
         new_stream_bank->SetVertexStream(source_stream.semantic(),
                                          source_stream.semantic_index(),
                                          source_fields[ii],
                                          0);
@@ skipping 68 matching lines @@
             // Bind streams
             skin_eval->SetVertexStream(source_stream.semantic(),
                                        source_stream.semantic_index(),
                                        source_field,
                                        0);
             new_stream_bank->BindStream(skin_eval,
                                         source_stream.semantic(),
                                         source_stream.semantic_index());
             break;
           }
+          default:
+            // do nothing
+            break;
         }
       }
       if (!copied) {
         Field* source_field = source_fields[ii];
         source_field->Copy(field);
       }
     }
 
     // Set all primitives to use new stream bank.
     for (unsigned ii = 0; ii < elements.size(); ++ii) {
@@ skipping 794 matching lines @@
         }
         case FUDaePassStateFaceType::BACK: {
           cull_front_ = false;
           break;
         }
         case FUDaePassStateFaceType::FRONT_AND_BACK: {
           O3D_ERROR(service_locator_)
               << "FRONT_AND_BACK culling is unsupported";
           break;
         }
+        case FUDaePassStateFaceType::INVALID:
+          O3D_ERROR(service_locator_) << "INVALID culling type";
+          break;
       }
       UpdateCullingState(state);
       break;
     }
     case FUDaePassState::DEPTH_FUNC: {
       State::Comparison function = ConvertComparisonFunction(
           service_locator_,
           GetStateValue<FUDaePassStateFunction::Function>(pass_state, 0));
       SetIntState(state, State::kZComparisonFunctionParamName, function);
       break;
@@ skipping 171 matching lines @@
     case FUDaePassState::LINE_SMOOTH_ENABLE: {
       bool value = GetStateValue<bool>(pass_state, 0);
       SetBoolState(state, State::kLineSmoothEnableParamName, value);
       break;
     }
     case FUDaePassState::STENCIL_TEST_ENABLE: {
       bool value = GetStateValue<bool>(pass_state, 0);
       SetBoolState(state, State::kStencilEnableParamName, value);
       break;
     }
+    default:
+      // do nothing
+      break;
   }
 }
 
 // Converts an FCollada texture sampler wrap mode to an O3D Sampler
 // AddressMode.
 static Sampler::AddressMode ConvertSamplerAddressMode(
     FUDaeTextureWrapMode::WrapMode wrap_mode) {
   switch (wrap_mode) {
     case FUDaeTextureWrapMode::WRAP:
       return Sampler::WRAP;
@@ skipping 158 matching lines @@
   size_t pcount = material->GetEffectParameterCount();
   // TODO(o3d):  This test (for determining if we used the
   // programmable profile or the fixed-func profile) is not very robust.
   // Remove this once the Material changes are in.
   if (pcount > 0) {
     for (size_t i = 0; i < pcount; ++i) {
       FCDEffectParameter* p = material->GetEffectParameter(i);
       LOG_ASSERT(p);
       String param_name(p->GetReference());
       // Check for an effect binding
-      FCDEffect* effect = material->GetEffect();
       FCDEffectProfileFX* profile_fx = FindProfileFX(material->GetEffect());
       if (profile_fx) {
         FCDEffectTechnique* technique = profile_fx->GetTechnique(0);
         if (technique->GetPassCount() > 0) {
           FCDEffectPass* pass = technique->GetPass(0);
           for (size_t j = 0; j < pass->GetShaderCount(); ++j) {
             FCDEffectPassShader* shader = pass->GetShader(j);
             FCDEffectPassBind* bind =
                 shader->FindBindingReference(p->GetReference());
             if (bind) {
@@ skipping 43 matching lines @@
       SetParamFromFCEffectParam(param_object,
                                 kMaterialParamNameShininess,
                                 effect_standard->GetShininessParam());
       SetParamFromFCEffectParam(param_object,
                                 kMaterialParamNameSpecularFactor,
                                 effect_standard->GetSpecularFactorParam());
     }
   }
 }
 }  // namespace o3d