Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(644)

Unified Diff: mojo/public/bindings/generators/cpp_templates/module.cc.tmpl

Issue 198343002: Mojo: request/response bindings (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/public/bindings/generators/cpp_templates/module.cc.tmpl
diff --git a/mojo/public/bindings/generators/cpp_templates/module.cc.tmpl b/mojo/public/bindings/generators/cpp_templates/module.cc.tmpl
index 2f017a045fb90bda2b50262213d85c5965615718..1c530c4f22e7e94bee51d88305936ddf4c3611d3 100644
--- a/mojo/public/bindings/generators/cpp_templates/module.cc.tmpl
+++ b/mojo/public/bindings/generators/cpp_templates/module.cc.tmpl
@@ -2,6 +2,11 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#if defined(__clang__)
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wunused-private-field"
+#endif
+
#include "{{module.path}}.h"
#include "mojo/public/bindings/lib/bindings_serialization.h"
@@ -12,20 +17,21 @@ namespace internal {
namespace {
#pragma pack(push, 1)
-#if defined(__clang__)
-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Wunused-private-field"
-#endif
+
{#--- Interface parameter definitions #}
{%- for interface in interfaces %}
{%- for method in interface.methods %}
-{%- set struct = interface|struct_from_method(method) %}
+{%- set method_name = "k%s_%s_Name"|format(interface.name, method.name) %}
+const uint32_t {{method_name}} = {{method.ordinal}};
+{% set struct = interface|struct_from_method(method) %}
{%- include "params_definition.tmpl" %}
+{%- if method.response_parameters != None %}
+{%- set struct = interface|response_struct_from_method(method) %}
+{%- include "params_definition.tmpl" %}
+{%- endif %}
{%- endfor %}
{%- endfor %}
-#if defined(__clang__)
-#pragma clang diagnostic pop
-#endif
+
#pragma pack(pop)
} // namespace
@@ -52,3 +58,7 @@ namespace {
{%- include "interface_definition.tmpl" %}
{%- endfor %}
} // namespace {{namespace}}
+
+#if defined(__clang__)
+#pragma clang diagnostic pop
+#endif

Powered by Google App Engine
This is Rietveld 408576698