[OpenSER-Devel] SF.net SVN: openser: [4067] trunk/modules/presence

[Anca Vamanu]

Hi Henning,

You are somehow right, the parameter was a bit useless. The idea was 
that this log will generate a lot of messages. But if not wanting them, 
the log level can be set to show only errors, so I've removed it.

Anca

Henning Westerholt wrote:
> On Tuesday 22 April 2008, Anca Vamanu wrote:
>   
>> Log Message:
>> -----------
>> - added an optional info log message for every Notify sent out; it can be
>> enabled from the script.
>>     
>
> Hello Anca,
>
> was there any special reason that you introduced a new module parameter to 
> trigger this one log message? I mean, LM_INFO is exactly meant for this type 
> of messages, informal and non critical stuff.. There are many modules in the 
> server that does similar things.
>
> I think its important to maintain a clear and concise module interface.
>
> Cheers,
>
> Henning
>