mojo/public/tools/bindings/mojom_bindings_generator.gypi - Issue 294833002: Mojo: more idiomatic C++ bindings

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: mojo/public/tools/bindings/mojom_bindings_generator.gypi

Issue 294833002: Mojo: more idiomatic C++ bindings (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: improvements per review feedback Created 6 years, 7 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:

Index: mojo/public/tools/bindings/mojom_bindings_generator.gypi

diff --git a/mojo/public/tools/bindings/mojom_bindings_generator.gypi b/mojo/public/tools/bindings/mojom_bindings_generator.gypi

index e325a9017a885c6c12840ae65ca005d86fce9426..a90f1ee39c25a8019c143ff5c1789a7eca598dc1 100644

--- a/mojo/public/tools/bindings/mojom_bindings_generator.gypi

+++ b/mojo/public/tools/bindings/mojom_bindings_generator.gypi

@@ -18,7 +18,6 @@

'inputs': [

'<(mojom_bindings_generator)',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/enum_declaration.tmpl',

- '<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/enum_traits.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/interface_declaration.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/interface_definition.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/interface_macros.tmpl',

@@ -30,12 +29,13 @@

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/module.h.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/module-internal.h.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/params_definition.tmpl',

- '<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/struct_builder_definition.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/struct_declaration.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/struct_definition.tmpl',

- '<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/struct_destructor.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/struct_macros.tmpl',

+ '<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/struct_serialization_declaration.tmpl',

+ '<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/struct_serialization_definition.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/wrapper_class_declaration.tmpl',

+ '<(DEPTH)/mojo/public/tools/bindings/generators/cpp_templates/wrapper_class_definition.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/js_templates/enum_definition.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/js_templates/interface_definition.tmpl',

'<(DEPTH)/mojo/public/tools/bindings/generators/js_templates/module.js.tmpl',