Re: [PATCH] header_wrappers.py generates 'return' for 'void' callback

On Fri, 29 Dec 2006, Kouhei Sutou wrote:
...
> [[
> Do not generate 'return' for callbacks return 'void'.
>
> * build/generator/swig/header_wrappers.py
> (_write_callback): Add 'return' only if return_type is not 'void'.
> ]]
...
> Index: build/generator/swig/header_wrappers.py
> ===================================================================
> --- build/generator/swig/header_wrappers.py (revision 22832)
> +++ build/generator/swig/header_wrappers.py (working copy)
> @@ -84,11 +84,15 @@
> param_names = string.join(self._re_param_names.findall(params), ", ")
> params = "%s _obj, %s" % (type, params)
>
> + invoke_callback = "%s(%s)" % (callee, param_names)
> + if return_type != "void":
> + invoke_callback = "return %s" % (invoke_callback)
                                         ^ ^
You don't need the parenthesis around (invoke_callback).

> # Write out the declaration
> self.ofile.write(
> "static %s %s_invoke_%s(\n" % (return_type, module, function) +
> " %s) {\n" % params +
> - " return %s(%s);\n" % (callee, param_names) +
> + " %s;\n" % (invoke_callback) +
> "}\n\n")

Other than that, looks good. +1 to commit.