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

Unified Diff: mojo/public/tools/bindings/generators/cpp_templates/interface_definition.tmpl

Issue 2112093002: Mojo C++ bindings: Merge EncodePointers/DecodePointers into Serialize/Deserialize, respectively. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@61_array_fix
Patch Set: . Created 4 years, 6 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/tools/bindings/generators/cpp_templates/interface_definition.tmpl
diff --git a/mojo/public/tools/bindings/generators/cpp_templates/interface_definition.tmpl b/mojo/public/tools/bindings/generators/cpp_templates/interface_definition.tmpl
index 4e4efce37a6acd7264960ea9d4f2d5b924a01259..b3c64233ff2b5286b7542cadeada98db6a9eff2a 100644
--- a/mojo/public/tools/bindings/generators/cpp_templates/interface_definition.tmpl
+++ b/mojo/public/tools/bindings/generators/cpp_templates/interface_definition.tmpl
@@ -7,7 +7,6 @@
{%- macro alloc_params(struct, params, message, serialization_context,
description) %}
- ({{params}})->DecodePointers();
({{serialization_context}})->handles.Swap(({{message}})->mutable_handles());
bool success = true;
{%- for param in struct.packed.packed_fields_in_ordinal_order %}
@@ -42,8 +41,6 @@ std::move(p_{{param.name}})
serialization_context)}}
({{serialization_context}})->handles.Swap(
builder.message()->mutable_handles());
- params->EncodePointers();
-
{%- endmacro %}
{#--- Begin #}

Powered by Google App Engine
This is Rietveld 408576698