[FFmpeg-devel] [PATCH] mpeg4video: Add support for MPEG-4 Simple Studio Profile.

Kieran Kunhya kieran618 at googlemail.com
Sun Jan 21 15:03:11 EET 2018

On Sun, Jan 21, 2018 at 12:53 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com>

> 2018-01-21 13:37 GMT+01:00 Kieran Kunhya <kieran618 at googlemail.com>:
> > On Mon, Jan 1, 2018 at 7:01 PM, Michael Niedermayer wrote:
> >
> >> Hi
> >
> > Patch updated.
> If you choose to ignore more than half of the comments made,
> please at least answer the review to explain why you
> believe the comments were not helpful.
> > Some of the review comments I decided not to implement
> > in order to keep closer to the spec.
> We have agreed in the past (on trac) that specifications
> are not necessarily written with the intention to allow
> writing good implementations, so I am not convinced
> this is a sufficient argumentation.

Ok, well I only did not implement a couple.
The startcode one conflates studio start codes with mpeg4 start codes.
The show_bits one I'd rather keep it as per the spec.
The n&1 comment the compiler should be able to deal with and not speed

The patch does not apply (and is not trivial to fix), Carl Eugen

Patch depends on
[PATCH] simple_idct: Template functions to support an input bitdepth


More information about the ffmpeg-devel mailing list