On Sun, Sep 4, 2016 at 6:31 PM, Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxxx> wrote: > > > On 04/09/16 11:54, Christian Couder wrote: >> On Thu, Sep 1, 2016 at 10:19 AM, Christian Couder >> <christian.couder@xxxxxxxxx> wrote: >>> On Thu, Sep 1, 2016 at 12:20 AM, Stefan Beller <sbeller@xxxxxxxxxx> wrote: >>>>> >>>>> +static void mute_routine(const char *bla, va_list params) >>>> >>>> Instead of 'bla' you could go with 'format' as the man page for >>>> [f]printf puts it. >>>> Or you could leave it empty, i.e. >>>> >>>> static void mute_routine(const char *, va_list) >>>> ... >>> >>> Ok to do that. >> >> Actually I get the following error when doing that: >> >> apply.c: In function ‘mute_routine’: >> apply.c:115:1: error: parameter name omitted >> static void mute_routine(const char *, va_list) >> ^ >> apply.c:115:1: error: parameter name omitted >> make: *** [apply.o] Error 1 > > Yes, this is not C++. ;-) > >> So I will leave it as is. > > I think I would prefer to see: > > static void mute_routine(const char *msg, va_list params) > > given that it would either be an error-msg or a warning-msg. Ok for "msg".