Make DEBUG macro invocations variadic

Use a script to update DEBUG macro invocations to use it as a variadic
macro, supplying format string and its arguments directly, instead of
wrapping them in parens.

This script was used to update the code:

grep -rwl --include '*.[hc]' DEBUG . |
    while read f; do
        mv "$f"{,.orig}
        perl -e \
            'use strict;
             use File::Slurp;
             my $text=read_file(\*STDIN);
             $text=~s#(\bDEBUG\s*\([^(]+)\((.*?)\)\s*\)\s*;#$1$2);#gs;
             print $text;' < "$f.orig" > "$f"
        rm "$f.orig"
    done

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Stephen Gallagher <sgallagh@redhat.com>
Reviewed-by: Simo Sorce <simo@redhat.com>

1 files changed, 4 insertions, 4 deletions

diff --git a/src/providers/ldap/ldap_id_services.c b/src/providers/ldap/ldap_id_services.c
index 04de1dbc4..77215127b 100644
--- a/src/providers/ldap/ldap_id_services.c
+++ b/src/providers/ldap/ldap_id_services.c
@@ -94,7 +94,7 @@ services_get_send(TALLOC_CTX *mem_ctx,
 
     state->op = sdap_id_op_create(state, state->conn->conn_cache);
     if (!state->op) {
-        DEBUG(SSSDBG_MINOR_FAILURE, ("sdap_id_op_create failed\n"));
+        DEBUG(SSSDBG_MINOR_FAILURE, "sdap_id_op_create failed\n");
         ret = ENOMEM;
         goto error;
     }
@@ -136,13 +136,13 @@ services_get_send(TALLOC_CTX *mem_ctx,
     talloc_zfree(clean_protocol);
     if (!state->filter) {
         DEBUG(SSSDBG_MINOR_FAILURE,
-              ("Failed to build the base filter\n"));
+              "Failed to build the base filter\n");
         ret = ENOMEM;
         goto error;
     }
     DEBUG(SSSDBG_TRACE_LIBS,
-          ("Preparing to search for services with filter [%s]\n",
-           state->filter));
+          "Preparing to search for services with filter [%s]\n",
+           state->filter);
 
     ret = build_attrs_from_map(state, id_ctx->opts->service_map,
                                SDAP_OPTS_SERVICES, NULL,