lists.zerezo.com



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [MPlayer-dev-eng] [PATCH] vo_macosx: use mmap instead of shmat for MPlayerOSX



On Mon, Oct 13, 2008 at 21:07:13 +0800, Ulion <ulion2002@xxxxxxxxx> wrote:

> it looks not bad, I have two questions:
>
> 1. although the same problem may also exists in current code, should
> the shm_id be unlinked when any error occurs after shm_open?
> 2. will this work on 10.3.*?
>

1. yes, that's right. else it won't be able to re-initialize the shared
memory after a failure, changed patches attached.
i also changed perror to mp_msg since that seems to be preferred and removed
shm_unlink from the mplayerosx code because it only has to be called from
mplayer (it would get unlinked from mplayerosx in the previous patch and
make mplayer fail on its later try).

2. shm_open/mmap have been around since 10.0, so it should theoretically
work on 10.3.
i also just confirmed that the patch works on 10.4. I don't have a 10.3 at
hand to test it there, though.

greetings
Adrian

Attachment: vo_macosx_shm.patch
Description: Binary data

Attachment: mplayerosx_shm.patch
Description: Binary data

_______________________________________________
MPlayer-dev-eng mailing list
MPlayer-dev-eng@xxxxxxxxxxxx
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng