[vlc-devel] Re: [PATCH] timestamp for log output and logger interface.

[Sigmund Augdal]

On Mon, Oct 25, 2004 at 08:39:25AM +0200, Philippe Van Hecke wrote:
> On Sunday 24 October 2004 18:59, Clément Stenac wrote:
> > Hello,
> >
> > > Any comment and review about it are welcome. And if it can be applied i
> > > will be more than happy :-)
> >
> > That can be a nice feature, and I don't think I have any comment on the
> > code (is localtime_r portable ?)
> 
> Looking a man pages of time functions i think.
> 
> >
> > But I would prefer to have this as a config option, and choose to enable
> > or disable the timestamp.
> 
> When i make de patch i thinking about this but don't really know where to put 
> this config argument and wich name to use for it. If you have an idea i can 
> review the patch in this way. 

Take a look at how this is done for the "time" module recently added.


Sigmund

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html