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

Clément Bœsch u at pkh.me
Thu Aug 21 07:34:50 CEST 2014


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.

> i think an API is only really available and stable after 1 revission

Yes, I'm sorry about that mistake, but the code was pushed only very
recently. What do you suggest?

> 
> rest LGTM
> 
[...]

-- 
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/83259837/attachment.asc>


More information about the ffmpeg-devel mailing list