[FFmpeg-devel] [PATCH] PGS subtitle position

[Carl Eugen Hoyos]

Mark Goodman <mark.goodman <at> gmail.com> writes:

> Position is present in the presentation segment when the object
> number is not zero.

[...]

> -    block = bytestream_get_byte(&buf);;
> -    if (block == 0x80) {

Any reason you remove this?
(I mean: Is this related to the PGS subtitle position fix or just an unrelated
improvement in which case it should be sent separately.)

> -         *     palette_update_flag (0x80),
> -         *     palette_id_to_use,

...

> +     *     palette_update_flag (0x80),
> +     *     palette_id_to_use

(some more to follow)

Please do not re-indent together with functional changes in one patch (make the
patch as small as possible, re-indentation should be done in a separate patch).

Carl Eugen