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

[Henning Westerholt]

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