[FFmpeg-devel] [PATCH] avutil/motion_vector.h: fix coordinate types

Clément Bœsch u at pkh.me
Thu Aug 21 12:37:33 CEST 2014


On Thu, Aug 21, 2014 at 11:42:53AM +0200, Michael Niedermayer wrote:
> On Thu, Aug 21, 2014 at 09:32:29AM +0200, Clément Bœsch wrote:
> > On Thu, Aug 21, 2014 at 07:34:50AM +0200, Clément Bœsch wrote:
> > > On Thu, Aug 21, 2014 at 12:01:51AM +0200, Michael Niedermayer wrote:
> > > > On Wed, Aug 20, 2014 at 11:30:27PM +0200, Clément Bœsch wrote:
> > > > > ---
> > > > >  doc/APIchanges            | 2 +-
> > > > >  libavutil/motion_vector.h | 8 ++++----
> > > > >  libavutil/version.h       | 2 +-
> > > > >  3 files changed, 6 insertions(+), 6 deletions(-)
> > > > > 
> > > > > diff --git a/doc/APIchanges b/doc/APIchanges
> > > > > index 1bed107..1fbeb09 100644
> > > > > --- a/doc/APIchanges
> > > > > +++ b/doc/APIchanges
> > > > > @@ -15,7 +15,7 @@ libavutil:     2014-08-09
> > > > >  
> > > > >  API changes, most recent first:
> > > > >  
> > > > > -2014-08-xx - xxxxxxx - lavu 54.5.100 - frame.h motion_vector.h
> > > > > +2014-08-xx - b0352b1,xxxxxxx - lavu 54.6.100 - frame.h motion_vector.h
> > > > 
> > > > what is "b0352b1,xxxxxxx" ?
> > > > 
> > > 
> > > b0352b1 is the main commit, at 54.5.100
> > > xxxxxxx would be this "fixup" commit, at 54.6.100
> > > 
> > > We wouldn't really want the users to use the feature in 54.5.100 because
> > > of the bug I'm fixing, but "b0352b1 - lavu 54.6.100" would leave a version
> > > inconsistency. And OTOH referencing only the new hash doesn't give much
> > > insight on the feature/change mentioned.
> > > 
> > 
> > One solution could be to clearly reference b0352b1 into the new commit,
> > and make sure we reference the new hash in the 54.6 in APIChanges.
> > 
> > Would that be fine?
> 
> yes
> 

Thanks, applied

> also to clarify what my concern was, it was for that listing 2
> hashes equally is confusing to a reader, we often have 2 hashes
> where one is from libav and one fro ffmpeg but i think this variant
> is new. Also it could scripts
[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140821/4a5ea803/attachment.asc>


More information about the ffmpeg-devel mailing list