[FFmpeg-devel] [FFMPEG] [PATCH] cavs encoder

[Stefan Gehrer]

zhihang wang wrote:
> Yes, you are right, Stefan.
> This is one of the problem. I have written the code as you just suggested
> before. But it will have to change the cavs decoder because the interfaces
> of some functions should be changed. So I rewrite some similar functions. I
> don't know whether these functions should be kept.
> This attachment is a new version of the cavs encoder. I change the code
> mainly according to* Diego's comments*. I rearrange the main part of the
> patch, cavsenc.c, carefully and added the comments for almost every function
> in detail in order to make the review easier.
> I would like to answer any questions on this patch and welcome anyone to
> review the code. I also like to make any change to let the patch be
> accepted.
> Another note should be given: The cavs decoder in the ffmpeg project now
> doesn't support the newest cavs version. So I think it should be updated.

Yes, I guess such things have to be expected if an already published
specification is changed in a way which is not backwards compatible.
I am aware of that one additional marker_bit before the picture_distance
element in I picture headers. Are there any more - maybe more serious -
incompatibilities?

Best regards
Stefan