Change command buffer client code to use structures....

core/cross/command_buffer/stream_bank_cb.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 129 matching lines @@
 
 // Creates the vertex struct resource on the service side. It will only set the
 // vertex inputs if they represent semantics and types we know about. The
 // command buffer API will not draw with an incomplete vertex struct.
 // This function will get called on Draw, after any change to the vertex inputs
 // has occurred.
 void StreamBankCB::CreateVertexStruct() {
   DCHECK_EQ(kInvalidResource, vertex_struct_id_);
   vertex_struct_id_ = renderer_->vertex_structs_ids().AllocateID();
   CommandBufferHelper *helper = renderer_->helper();
-  CommandBufferEntry args[5];
-  args[0].value_uint32 = vertex_struct_id_;
-  args[1].value_uint32 = vertex_stream_params_.size();
-  helper->AddCommand(command_buffer::CREATE_VERTEX_STRUCT, 2, args);
+  helper->CreateVertexStruct(vertex_struct_id_, vertex_stream_params_.size());
   for (unsigned int i = 0; i < vertex_stream_params_.size(); ++i) {
     const Stream &stream = vertex_stream_params_[i]->stream();
     vertex_struct::Semantic cb_semantic;
     unsigned int cb_semantic_index;
     if (!GetCBSemantic(stream.semantic(), stream.semantic_index(), &cb_semantic,
                        &cb_semantic_index)) {
       DLOG(INFO) << "Unknown semantic (" << stream.semantic() << ", "
                  << stream.semantic_index() << ") - ignoring stream.";
       continue;
     }
     vertex_struct::Type cb_type = GetCBType(stream.field());
     if (cb_type == vertex_struct::NUM_TYPES) {
       DLOG(INFO) << "Invalid type (" << stream.field().num_components()
                  << ") - ignoring stream.";
       continue;
     }
 
-    namespace cmd = command_buffer::set_vertex_input_cmd;
     VertexBufferCB *vertex_buffer =
         static_cast<VertexBufferCB *>(stream.field().buffer());
-    args[0].value_uint32 = vertex_struct_id_;
-    args[1].value_uint32 = i;
-    args[2].value_uint32 = vertex_buffer->resource_id();
-    args[3].value_uint32 = stream.field().offset();
-    args[4].value_uint32 =
-        cmd::SemanticIndex::MakeValue(cb_semantic_index) |
-        cmd::Semantic::MakeValue(cb_semantic) |
-        cmd::Type::MakeValue(cb_type) |
-        cmd::Stride::MakeValue(vertex_buffer->stride());
-    helper->AddCommand(command_buffer::SET_VERTEX_INPUT, 5, args);
+    helper->SetVertexInput(
+        vertex_struct_id_, i,
+        vertex_buffer->resource_id(),
+        stream.field().offset(),
+        cb_semantic,
+        cb_semantic_index,
+        cb_type,
+        vertex_buffer->stride());
   }
 }
 
 // Destroys the vertex struct resource on the service side.
 void StreamBankCB::DestroyVertexStruct() {
   if (vertex_struct_id_ != kInvalidResource) {
     CommandBufferHelper *helper = renderer_->helper();
-    CommandBufferEntry args[1];
-    args[0].value_uint32 = vertex_struct_id_;
-    helper->AddCommand(command_buffer::DESTROY_VERTEX_STRUCT, 1, args);
+    helper->DestroyVertexStruct(vertex_struct_id_);
     renderer_->vertex_structs_ids().FreeID(vertex_struct_id_);
     vertex_struct_id_ = kInvalidResource;
   }
 }
 
 void StreamBankCB::BindStreamsForRendering() {
   if (vertex_struct_id_ == kInvalidResource)
     CreateVertexStruct();
   CommandBufferHelper *helper = renderer_->helper();
-  CommandBufferEntry args[6];
-  // Sets current vertex struct.
-  args[0].value_uint32 = vertex_struct_id_;
-  helper->AddCommand(command_buffer::SET_VERTEX_STRUCT, 1, args);
+  helper->SetVertexStruct(vertex_struct_id_);
 }
 
 }  // namespace o3d