[vlc-devel] [PATCH 2/2] vlc_fixups: define EPROTO if necessary on OS/2

[KO Myung-Hun]

Rémi Denis-Courmont wrote:
> Le torstaina 29. syyskuuta 2016, 15.34.48 EEST KO Myung-Hun a écrit :
>> ---
>>  include/vlc_fixups.h | 5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/include/vlc_fixups.h b/include/vlc_fixups.h
>> index 7a76ef4..f87179d 100644
>> --- a/include/vlc_fixups.h
>> +++ b/include/vlc_fixups.h
>> @@ -473,6 +473,11 @@ struct addrinfo
>>  };
>>
>>  void freeaddrinfo (struct addrinfo *res);
>> +
>> +# include <errno.h>
>> +# ifndef EPROTO
>> +#  define EPROTO (ELAST + 1)
>> +# endif
>>  #endif
>>
>>  /* math.h */
> 
> Not sure going beyond ELAST is a good idea. Isn´t there a sys_errlist[] or 
> equivalent on OS/2?
> 

I've searched for the fixup of EPROTO for a time, especially in GNULIB.

GNULIB uses two strategy. One is to define EPROTO to the value(134) used
by upward versions on Windows. The other is to define to arbitrary
value(2003) not used any other errno value on any other platforms.

http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=blob;f=lib/errno.in.h;h=96503ecac2e7979241c2507b80d7af2f9adfce36;hb=HEAD

OS/2 does not have EPROTO support at all, and equivalent ones too.
Thus, it's not possible to define the value used by upward versions.
After all, to define beyond ELAST has no problem.

-- 
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