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

Michael Niedermayer michaelni at gmx.at
Thu Aug 21 11:42:53 CEST 2014


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

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



[...]


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140821/a0dcc7dc/attachment.asc>


More information about the ffmpeg-devel mailing list