On 09/06/2013 07:39 PM, Anoop Saldanha wrote:
> In src/util-misc.h:
> 
>> @@ -24,6 +24,22 @@
>>  #ifndef __UTIL_MISC_H__
>>  #define __UTIL_MISC_H__
>>  
>> +#include "util-error.h"
>> +
>> +/**
>> + * \brief Generic API that can be used by all to log an
>> + *        invalid conf entry.
>> + * \param param_name A string specifying the param name.
>> + * \param value Default value to be printed.
>> + * \param format Format for above value.  For example "%s", "%"PRIu32, etc.
>> + */
>> +#define WarnInvalidConfEntry(param_name, value, format) do {            \
> 
> Yeah to the format part. I realized this when I made the commit, but I
> kept it because I want the value comment in the description above the
> format. Will change it back.
> 
> Kept it as a part of misc, in case we decide to have other such logging
> convenience macros. What do you suggest?

I've accepted it in misc in the new pr. It's both logging and conf, so
misc sounds good :)