[FFmpeg-cvslog] h264_mp4toannexb_bsf: support 24bit lengths

Michael Niedermayer michaelni at gmx.at
Wed Mar 21 22:59:22 CET 2012


On Wed, Mar 21, 2012 at 10:08:55PM +0100, Reimar Döffinger wrote:
> On Wed, Mar 21, 2012 at 10:04:53PM +0100, gs_gail wrote:
> > -        if (ctx->length_size == 1) {
> > -            nal_size = buf[0];
> > -        } else if (ctx->length_size == 2) {
> > -            nal_size = AV_RB16(buf);
> > -        } else
> > -            nal_size = AV_RB32(buf);
> > +        for(nal_size = 0, unit_type = 0; unit_type<ctx->length_size; unit_type++)
> > +            nal_size = (nal_size << 8) | buf[unit_type];
> 
> I'd like to request an extra variable like "i" instead of mis-/reusing
> unit_type.

fixed

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20120321/1ef70373/attachment.asc>


More information about the ffmpeg-cvslog mailing list