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

[Sigmund Augdal]

On Mon, Oct 25, 2004 at 01:28:03PM +0200, Philippe Van Hecke wrote:
> > Take a look at how this is done for the "time" module recently added.
> >
> >
> > Sigmund
> Hi,
> 
> I have take a look into the time modules. But some questions remains. 
> 
> 1) Can i put the FormatTime somewhere so it can be shared by others modules.
I guess you could put it in src/misc/mtime.c or where the current time
functions are located. (make sure to add a matching VLC_EXPORT in a relevant
header file).

> 
> 2) Should i need to define a  --logger-time-format into the logger modules 
> or --time-format argument can be defined globaly and and shared by other 
> modules ?
I think the requirements for timestamps in log files and for printing time
on top of video are much different. Basically I'd want logfile timestamps to
be compact and easily parsable while the video merged timestams should be
easily human readable, and constant width (to keep them from bouncing
about). So I think you should keep the options separated.

Sigmund

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