[MPlayer-dev-eng] [PATCH] vobsub missing subs

Ivan Kalvachev ikalvachev at gmail.com
Thu Apr 22 11:47:37 CEST 2010


On 4/18/10, ubitux <ubitux at gmail.com> wrote:
> According to what Uoti Urpala said to me on IRC, it would be a problem to
> keep static state variable in case of having many instances of the parser
> in parallel.
>
> So here is a second version of the patch using the mpeg context struct.



On 4/14/10, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> ubitux <ubitux <at> gmail.com> writes:
>
>> In fact, it seems that those packets are simply not padded with padding
>> packets. So here is a patch which seems to work to handle the issue.
>
> Are the two variables needed?
>
>> As the issue is fixed, I just removed a dead code in spudec which was
>> another hack to partially fix the issue.
>
> Applied. Please try to keep issues separated.

It seems there is a little bit of confusion here.
I thought the "applied" above is about the patch, but it seems it is
about the old hack removal.


Anyway, as there is new patch I would give it 2 more days deadline.
If nobody objects and nobody finds any bugs and not working streams until then,
I'll commit it.

Anybody who is more confident than me, is free to commit is sooner.



More information about the MPlayer-dev-eng mailing list