[FFmpeg-devel] [PATCH 3/3] lavc: implement an ATRAC9 decoder

Rostislav Pehlivanov atomnuker at gmail.com
Tue Jul 3 22:19:56 EEST 2018


On 3 July 2018 at 18:28, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:

> 2018-07-03 17:43 GMT+02:00, Rostislav Pehlivanov <atomnuker at gmail.com>:
> > On 1 July 2018 at 13:27, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
> >
> >> 2018-07-01 13:45 GMT+02:00, Rostislav Pehlivanov <atomnuker at gmail.com>:
> >>
> >> > Here's the last patch which fixes the lack of timestamps. Seems
> >> > like what's done for atrac3p is enough.
> >>
> >> You should bump some version number for the new feature, I believe
> >> we have always bumped minor for new codecs, you could merge
> >> patches one and three.
> >>
> > I did bump micro but I'll change the bump to minor.
>
> My suggestion was to bump minor when adding the decoder (which
> is the only thing a library user may be interested in) but to simplify,
> just merge 1 and 3.
>
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

Oh, right.
Pushed after verifying the output of the decoder matches the official
binary decoder, merged 1 and 3 (with a minor bump instead of micro).
Thanks.


More information about the ffmpeg-devel mailing list