[MPlayer-dev-eng] [PATCH] forked code may not use GUI
A Mennucc
mennucc1 at debian.org
Sat Oct 6 18:28:38 CEST 2007
hi
there was a discussion long ago regarding bug
http://bugs.debian.org/396962
at that time, Reimar proposed a simple and effective patch
(see message below) but that
unfortunately would break mencode compilation
I have revised his idea, see attached patch
a.
Reimar Döffinger ha scritto:
> Hello,
> On Sat, Jan 27, 2007 at 11:15:30AM +0100, Reimar D?ffinger wrote:
>> On Sat, Jan 27, 2007 at 10:30:30AM +0100, A Mennucc wrote:
>>> So the reason that my patch for threaded caching works is that it is
>>> possible to open dialog windows from different threads.
>>>
>>> possible solutions:
>>>
>>> 1) make sure you do not ever ever display a dialog from the caching code
>>> 2) use pthreads
>>>
>>> I would suggest the 2nd :-)
>>> (as I explained in my other email, mplayer is already compiled to use
>>> pthreads in linux)
>> 2) is not a solution to this problem it is a hack-around (or in German:
>> a Würgaround).
>> I think your suggestion has merits, but this problem is not one of them!
>> Actually, I consider hiding a bug a point against a patch.
>
> Except that it breaks mencoder linking, attached patch is closer to a
> proper fix for that problem.
>
> Greetings,
> Reimar Döffinger
>
(I deleted the attached patch of Reimar to avoid confusion - what
follows is mine)
a.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fork-and-gui.diff
Type: text/x-patch
Size: 1022 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20071006/9ee60d6c/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 252 bytes
Desc: OpenPGP digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20071006/9ee60d6c/attachment.pgp>
More information about the MPlayer-dev-eng
mailing list