[vlc-devel] [PATCH 6/6] misc: use socketpair() instead of vlc_pipe() on OS/2

KO Myung-Hun komh78 at gmail.com
Tue Jul 7 04:59:11 CEST 2015



Rémi Denis-Courmont wrote:
> Le lundi 06 juillet 2015, 17:03:25 KO Myung-Hun a écrit :
>> OS/2 vlc_poll() does not accept non-socket fds.
>> ---
>>  src/misc/interrupt.c | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/src/misc/interrupt.c b/src/misc/interrupt.c
>> index 7ec7fa9..37ef7a3 100644
>> --- a/src/misc/interrupt.c
>> +++ b/src/misc/interrupt.c
>> @@ -298,7 +298,11 @@ static int vlc_poll_i11e_inner(struct pollfd *restrict
>> fds, unsigned nfds, fd[1] = fd[0];
>>      else
>>  # endif
>> +#ifndef __OS2__
>>      if (vlc_pipe(fd))
>> +#else
>> +    if (socketpair(AF_LOCAL, SOCK_STREAM, 0, fd))
>> +#endif
>>      {
>>          vlc_testcancel();
>>          errno = ENOMEM;
> 
> I was first tempted to ask to merge the endif and ifndef. But then I remembered 
> that if pipes need a way to be polled anyhow. Otherwise, you are hosed.
> 

I'm sorry I don't understand what you are saying.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.7.2
Under OS/2 Warp 4 for Korean with FixPak #15
In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr




More information about the vlc-devel mailing list