> > +  static char const fmt[] = "Unknown error (%d)";
> 
> The only improvement I see is to change the string to something like
> "errno=%d", since it won't be translated to a native language.  That's
> good enough for a substitute function.

That string was the same as what was used in the missing strerror
case, so for now, I committed the patch as-is.  We can reword
it later, if needed (does Interix even support locales)?

But come to think of it, now that we assume C89, should we
simplify the code by not even catering to the missing strerror
case?  Which modern platforms still lack strerror?

-- 
Eric Blake

-- 
View this message in context: 
http://www.nabble.com/Re%3A-error.c%3A-%22Unknown-system-error%22-should-report-errno-value-tf4513158.html#a12884133
Sent from the Gnulib mailing list archive at Nabble.com.



Reply via email to