[vlc-devel] [PATCH 3/3] libvlc: use strerror_r() on Solaris

[Sean McGovern]

On Mon, Jan 5, 2015 at 1:32 PM, Jean-Baptiste Kempf <jb at videolan.org> wrote:

> On 03 Jan, Sean McGovern wrote :
> > On Sat, Jan 3, 2015 at 1:11 PM, Jean-Baptiste Kempf <jb at videolan.org>
> wrote:
> >
> > > On 03 Jan, Sean McGovern wrote :
> > > > Solaris libc does not implement strerror_l().
> > >
> > > Weird question, I know, but why not add this to compat?
> > >
> > > With my kindest regards,
> > >
> > > --
> > > Jean-Baptiste Kempf
> > > http://www.jbkempf.com/ - +33 672 704 734
> > > Sent from my Electronic Device
> >
> >
> > I could do that -- I was just following the way it was done for the other
> > operating systems like Android and OS/2.
>
> Android does it only because this is a Android-specific API, IIRC.
> OS/2 is a peculiar OS.
>
> > Let me know if you prefer that I move it.
>
> Yes.
>
> AC_REPLACE_FUNC is your friend.
>
>
Further investigation shows this as being unreasonable -- the function
signature for strerror_l is:

char *strerror_l(int errnum, locale_t loc);

Until POSIX 2008, the locale_t structure was non-standard and in my case,
on Solaris, does not exist. Neither does newlocale().

Making them up would just be absurd.

I don't know how to proceed from here -- but now I'm thinking strerror_l()
should not be assumed to be a replacement for strerror_r() given that it is
not widely implemented in vendor C libraries.

-- Sean McG.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20150112/d5126f2d/attachment.html>