[Ffmpeg-devel] Patch add function to read record time in dv video

Michael Niedermayer michaelni
Sun Sep 4 16:12:16 CEST 2005


Hi

On Sun, Sep 04, 2005 at 03:40:05PM +0200, Diego Biurrun wrote:
> On Sat, Sep 03, 2005 at 02:58:52PM +0200, Diego Biurrun wrote:
> > On Fri, Sep 02, 2005 at 11:35:42PM +0200, Michael Niedermayer wrote:
> > > 
> > > On Fri, Sep 02, 2005 at 11:23:48PM +0200, Diego Biurrun wrote:
> > > > On Fri, Sep 02, 2005 at 11:55:41AM -0600, Fred Rothganger wrote:
> > > > > 
> > > > > --- dv.c	2 Sep 2005 08:30:26 -0000	1.45
> > > > > +++ dv.c	2 Sep 2005 17:49:19 -0000
> > > > > @@ -67,7 +67,7 @@
> > > > > -     dv_viedo_recdate = 0x62,
> > > > > +     dv_video_recdate = 0x62,
> > > > > @@ -251,7 +251,7 @@
> > > > > -    case dv_viedo_recdate:  /* VAUX recording date */
> > > > > +    case dv_video_recdate:  /* VAUX recording date */
> > > > 
> > > > This is an unrelated fix for a stupid typo.
> > > > 
> > > > Michael, if you don't mind I'll commit this, OK?
> > > 
> > > ok, and please dont ask about such trivial things but instead just commit them
> > 
> > Applied and I won't bother you with these patches in the future.  Thanks
> > for the confidence.
> 
> And here is the updated patch without the typo fix for review...

patch looks ok, roman, do you agree with that? IIRC we had some
disagreements in the past about how to pass this info around

[...]

-- 
Michael





More information about the ffmpeg-devel mailing list