ppapi/generators/idl_gen_wrapper.py - Issue 305233002: PPAPI: Make "no_interface_string" APIs have no PNaCl wrapper

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: ppapi/generators/idl_gen_wrapper.py

Issue 305233002: PPAPI: Make "no_interface_string" APIs have no PNaCl wrapper (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: review fix 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: ppapi/generators/idl_gen_wrapper.py

diff --git a/ppapi/generators/idl_gen_wrapper.py b/ppapi/generators/idl_gen_wrapper.py

index 94889d6c5dd61965ffb36bda805a78e33e2c84e3..5d38eff8c9a92465d7e4ebfb200a769f61f382a9 100644

--- a/ppapi/generators/idl_gen_wrapper.py

+++ b/ppapi/generators/idl_gen_wrapper.py

@@ -349,9 +349,7 @@ const void *__%(wrapper_prefix)s_PPPGetInterface(const char *name) {

def GenerateWrapperInfoAndCollection(self, iface_releases, out):

for iface in iface_releases:

- iface_macro = "NULL"

- if not iface.node.GetProperty("no_interface_string"):

- iface_macro = self.cgen.GetInterfaceMacro(iface.node, iface.version)

+ iface_macro = self.cgen.GetInterfaceMacro(iface.node, iface.version)

if iface.needs_wrapping:

wrap_iface = '(const void *) &%s_Wrappers_%s' % (self.wrapper_prefix,

iface.struct_name)