[MPlayer-cvslog] CVS: main/input input.c,1.132,1.133

[Diego Biurrun]

On Thu, Mar 23, 2006 at 02:41:46PM +0100, Alban Bedel wrote:
> On Thu, 23 Mar 2006 10:34:02 +0100
> Diego Biurrun <diego at biurrun.de> wrote:
> 
> > On Thu, Mar 23, 2006 at 12:05:16AM +0100, Alban Bedel CVS wrote:
> > > 
> > > Modified Files:
> > > 	input.c 
> > > Log Message:
> > > Move audio_delay to a property and add a few read only properties to
> > > query samplerate, format, etc.
> > > 
> > > --- input.c	22 Mar 2006 00:19:02 -0000	1.132
> > > +++ input.c	22 Mar 2006 23:05:13 -0000	1.133
> > > @@ -51,7 +51,7 @@
> > >  #ifdef USE_EDL
> > >    { MP_CMD_EDL_MARK, "edl_mark", 0, { {-1,{0}} } },
> > >  #endif
> > > -  { MP_CMD_AUDIO_DELAY, "audio_delay", 1, { {MP_CMD_ARG_FLOAT,{0}}, {-1,{0}} } },
> > > +  { MP_CMD_AUDIO_DELAY, "audio_delay", 1, { {MP_CMD_ARG_FLOAT,{0}}, {MP_CMD_ARG_INT,{0}}, {-1,{0}} } },
> > 
> > What's this second argument?  Does it need to be documented?
> 
> audio_delay <value> [<abs>], set the audio delay if <abs> is non zero,
> adjust it otherwise. Yep it need to be documented, i have the changes
> in my dev tree but i was planning to commit them only when i'm finished
> merging all that code (only 1 or 2 merge left ;).

OK, please try not to leave code undocumented.

Diego