[FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

Clément Bœsch u at pkh.me
Sat Jun 25 14:15:56 CEST 2016


On Fri, Jun 24, 2016 at 09:20:35AM +0200, Benoit Fouet wrote:
[...]
> >>Any objection to this patch now?
> >iam ok with the patch, maybe give others a bit of time to reply
> >before applying though
> 
> Yeah, I'm in no hurry, I just saw this FIXME in the context of one of
> Mateo's patches.
> I was more waiting for some feedback from Hendrik or Clément, anyway, as
> they were the ones raising the points.

Should be fine if indeed the following computation:

        int width  = 16 * sps->mb_width;
        int height = 16 * sps->mb_height * (2 - sps->frame_mbs_only_flag);

...is always correct.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160625/a0cc9af9/attachment.sig>


More information about the ffmpeg-devel mailing list