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

Ivan Kalvachev ikalvachev at gmail.com
Wed Apr 28 00:08:36 CEST 2010


On 4/22/10, Ivan Kalvachev <ikalvachev at gmail.com> wrote:
> 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.
>

Committed.



More information about the MPlayer-dev-eng mailing list