[FFmpeg-devel] [PATCH] movenc: Fix muxing of Apple Quicktime chapters.

[Philip Langdale]

On Sun, 13 May 2012 17:33:41 -0700
Philip Langdale <philipl at overt.org> wrote:

> +    else {
> +        /* Stub header (usually for Quicktime chapter track) */
> +        // TextSampleEntry
> +        avio_wb32(pb, 0x01); // displayFlags
> +        avio_w8(pb, 0x00);   // horizontal justification
> +        avio_w8(pb, 0x00);   // vertical justification
> +        avio_w8(pb, 0x00);   // bgColourRed
> +        avio_w8(pb, 0x00);   // bgColourGreen
> +        avio_w8(pb, 0x00);   // bgColourBlue
> +        avio_w8(pb, 0x00);   // bgColourAlpha
> +        // BoxRecord
> +        avio_wb16(pb, 0x00); // defTextBoxTop
> +        avio_wb16(pb, 0x00); // defTextBoxLeft
> +        avio_wb16(pb, 0x00); // defTextBoxBottom
> +        avio_wb16(pb, 0x00); // defTextBoxRight
> +        // StyleRecord
> +        avio_wb16(pb, 0x00); // startChar
> +        avio_wb16(pb, 0x00); // endChar
> +        avio_wb16(pb, 0x01); // fontID
> +        avio_w8(pb, 0x00);   // fontStyleFlags
> +        avio_w8(pb, 0x00);   // fontSize
> +        avio_w8(pb, 0x00);   // fgColourRed
> +        avio_w8(pb, 0x00);   // fgColourGreen
> +        avio_w8(pb, 0x00);   // fgColourBlue
> +        avio_w8(pb, 0x00);   // fgColourAlpha
> +        // FontTableBox
> +        avio_wb32(pb, 0x0D); // box size
> +        ffio_wfourcc(pb, "ftab"); // box atom name
> +        avio_wb16(pb, 0x01); // entry count
> +        // FontRecord
> +        avio_wb16(pb, 0x01); // font ID
> +        avio_w8(pb, 0x00);   // font name length
> +    }

This stub definition comes straight from the 3gpp docs. From
a size point of view it matches the incorrect one from mpeg4ip,
so that w8 isn't wrong per se. It all lines up in the end.

--phil