[FFmpeg-devel] [PATCH] Bluray Subtitle Support

Aurelien Jacobs aurel
Thu Jul 23 22:15:43 CEST 2009


On Thu, Jul 23, 2009 at 07:16:21PM +1000, stev391 at exemail.com.au wrote:
> G'Day,
> 
> Please find attached a patch that adds Bluray subtitle support.
> 
> [...]
> 
> +    /* Get bitmap dimensions from data */
> +    width = AV_RB16(buf);
> +    buf += 2;
> +
> +    height = AV_RB16(buf);
> +    buf += 2;

Here you should use bytestream_get_be16() to simplify code.
This applies to many other parts of your code (basically every place
you use AV_R???()).

Aurel



More information about the ffmpeg-devel mailing list