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, 2 insertions, 2 deletions

diff --git a/src/providers/ldap/ldap_access.c b/src/providers/ldap/ldap_access.c
index c468a1cd5..05354d10c 100644
--- a/src/providers/ldap/ldap_access.c
+++ b/src/providers/ldap/ldap_access.c
@@ -69,7 +69,7 @@ void sdap_pam_access_handler(struct be_req *breq)
                            access_ctx->id_ctx->conn,
                            pd);
     if (req == NULL) {
-        DEBUG(SSSDBG_CRIT_FAILURE, ("Unable to start sdap_access request\n"));
+        DEBUG(SSSDBG_CRIT_FAILURE, "Unable to start sdap_access request\n");
         sdap_access_reply(breq, PAM_SYSTEM_ERR);
         return;
     }
@@ -97,7 +97,7 @@ static void sdap_access_done(struct tevent_req *req)
         pam_status = PAM_ACCT_EXPIRED;
         break;
     default:
-        DEBUG(SSSDBG_CRIT_FAILURE, ("Error retrieving access check result.\n"));
+        DEBUG(SSSDBG_CRIT_FAILURE, "Error retrieving access check result.\n");
         pam_status = PAM_SYSTEM_ERR;
         break;
     }