diff options
author | Lucas De Marchi <lucas.demarchi@profusion.mobi> | 2012-10-04 04:26:26 -0300 |
---|---|---|
committer | Johan Hedberg <johan.hedberg@intel.com> | 2012-10-04 17:05:22 +0300 |
commit | 312d738e0e4ad098c3087997df6c8e907f41fa42 (patch) | |
tree | a96a9630acda2b03e5418330bc097e8a088a7428 /gdbus/object.c | |
parent | 7084f0baf5cb73d7a3632233fe409bb4bd652210 (diff) | |
download | bluez-312d738e0e4ad098c3087997df6c8e907f41fa42.tar.gz |
gdbus: Use macros to add annotations
Besides being more readable this way it avoids going over 80 chars.
Diffstat (limited to 'gdbus/object.c')
-rw-r--r-- | gdbus/object.c | 40 |
1 files changed, 28 insertions, 12 deletions
diff --git a/gdbus/object.c b/gdbus/object.c index 96890064c..24e8285d8 100644 --- a/gdbus/object.c +++ b/gdbus/object.c @@ -76,6 +76,16 @@ static void print_arguments(GString *gstr, const GDBusArgInfo *args, } } +#define G_DBUS_ANNOTATE(prefix_, name_, value_) \ + prefix_ "<annotation name=\"org.freedesktop.DBus." name_ "\" " \ + "value=\"" value_ "\"/>\n" + +#define G_DBUS_ANNOTATE_DEPRECATED(prefix_) \ + G_DBUS_ANNOTATE(prefix_, "Deprecated", "true") + +#define G_DBUS_ANNOTATE_NOREPLY(prefix_) \ + G_DBUS_ANNOTATE(prefix_, "Method.NoReply", "true") + static void generate_interface_xml(GString *gstr, struct interface_data *iface) { const GDBusMethodTable *method; @@ -90,19 +100,22 @@ static void generate_interface_xml(GString *gstr, struct interface_data *iface) if (!deprecated && !noreply && !(method->in_args && method->in_args->name) && !(method->out_args && method->out_args->name)) - g_string_append_printf(gstr, "\t\t<method name=\"%s\"/>\n", - method->name); + g_string_append_printf(gstr, + "\t\t<method name=\"%s\"/>\n", + method->name); else { - g_string_append_printf(gstr, "\t\t<method name=\"%s\">\n", - method->name); + g_string_append_printf(gstr, + "\t\t<method name=\"%s\">\n", + method->name); print_arguments(gstr, method->in_args, "in"); print_arguments(gstr, method->out_args, "out"); if (deprecated) - g_string_append_printf(gstr, "\t\t\t<annotation name=\"org.freedesktop.DBus.Deprecated\" value=\"true\"/>\n"); - + g_string_append_printf(gstr, + G_DBUS_ANNOTATE_DEPRECATED("\t\t\t")); if (noreply) - g_string_append_printf(gstr, "\t\t\t<annotation name=\"org.freedesktop.DBus.Method.NoReply\" value=\"true\"/>\n"); + g_string_append_printf(gstr, + G_DBUS_ANNOTATE_NOREPLY("\t\t\t")); g_string_append_printf(gstr, "\t\t</method>\n"); } @@ -113,15 +126,18 @@ static void generate_interface_xml(GString *gstr, struct interface_data *iface) G_DBUS_SIGNAL_FLAG_DEPRECATED; if (!deprecated && !(signal->args && signal->args->name)) - g_string_append_printf(gstr, "\t\t<signal name=\"%s\"/>\n", - signal->name); + g_string_append_printf(gstr, + "\t\t<signal name=\"%s\"/>\n", + signal->name); else { - g_string_append_printf(gstr, "\t\t<signal name=\"%s\">\n", - signal->name); + g_string_append_printf(gstr, + "\t\t<signal name=\"%s\">\n", + signal->name); print_arguments(gstr, signal->args, NULL); if (deprecated) - g_string_append_printf(gstr, "\t\t\t<annotation name=\"org.freedesktop.DBus.Deprecated\" value=\"true\"/>\n"); + g_string_append_printf(gstr, + G_DBUS_ANNOTATE_DEPRECATED("\t\t\t")); g_string_append_printf(gstr, "\t\t</signal>\n"); } |