[OpenSER-Devel] SF.net SVN: openser: [3415] trunk

Henning Westerholt henning.westerholt at 1und1.de
Mon Dec 17 18:16:30 UTC 2007


On Monday 17 December 2007, Bogdan-Andrei Iancu wrote:
> Hi Henning,
>
> I think this needs a bit of a re-work as what it supposed to be in
> modules ended up in core. The solution is to have a generic LM_ function
> where you can pass facility as argument. So, the ACC facility will be a
> module parameter for acc module - the same as the "log_level". See:
>     http://www.openser.org/docs/modules/1.3.x/acc.html#AEN268
>
> So, my suggestion is to revert this commit and let me add this generic
> logging function (which anyhow is needed by other modules, like xlog)
> and to add the configurable facility as acc module param.

Hi Bogdan,

yes, this really belongs more to the module code. But this patch was getting 
really stale in my repository, and i do not wanted to extend the logging 
interface only for this module. I was also not aware the the xlog module 
needs the same interface extension.

I revert the commit later. 

Cheers,

Henning



More information about the Devel mailing list