CHROMIUM: provide default implementation for signal handlers
A D-Bus client may not be interested of all the D-Bus signals emitted by
a D-Bus server. This CL modifies the proxy template in dbus-c++ such
that the base proxy class provides a default implementation for signal
handlers, which can be overridden by a subclass as needed.
BUG=chromium:776452
TEST=Build cromo and wimax-manager.
TEST=Remote trybot runs.
Change-Id: I64057d31dc1784bddc29bbb02977fe9a29e1a81e
Reviewed-on: https://chromium-review.googlesource.com/917528
Commit-Ready: Ben Chan <[email protected]>
Tested-by: Ben Chan <[email protected]>
Reviewed-by: Dan Erat <[email protected]>
diff --git a/tools/proxy-stubs.tpl b/tools/proxy-stubs.tpl
index 0c0a922..3d49f7d 100644
--- a/tools/proxy-stubs.tpl
+++ b/tools/proxy-stubs.tpl
@@ -146,7 +146,7 @@
*/
{{#FOR_EACH_SIGNAL}}
- virtual void {{SIGNAL_NAME}}({{#CONST_SIGNAL_ARG_LIST}}{{SIGNAL_ARG_DECL}}{{#CONST_SIGNAL_ARG_LIST_separator}}, {{/CONST_SIGNAL_ARG_LIST_separator}}{{/CONST_SIGNAL_ARG_LIST}}) = 0;
+ virtual void {{SIGNAL_NAME}}({{#CONST_SIGNAL_ARG_LIST}}{{SIGNAL_ARG_DECL}}{{#CONST_SIGNAL_ARG_LIST_separator}}, {{/CONST_SIGNAL_ARG_LIST_separator}}{{/CONST_SIGNAL_ARG_LIST}}) {}
{{/FOR_EACH_SIGNAL}}
{{BI_NEWLINE}}protected: