bug-indent
[Top][All Lists]
Advanced

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

indent 2.2.11: HAVE_LCCTYPES (?)


From: Santiago Vila
Subject: indent 2.2.11: HAVE_LCCTYPES (?)
Date: Sun, 11 Apr 2010 16:49:43 +0200 (CEST)
User-agent: Alpine 1.10 (DEB 962 2008-03-14)

Hello.

Line 1011 in src/indent.c says:

#if defined (HAVE_SETLOCALE) && defined (HAVE_LC_MESSAGES) && defined 
(HAVE_LCCTYPES)
  setlocale(LC_MESSAGES, "");


Is this really supposed to work? Maybe you meant "HAVE_LC_CTYPE"?
In either case, there is no configure check for that so it will not work
anyway.

For reference, such line is supposed to fix this bug:

http://bugs.debian.org/205692

This is what I did in the Debian indent package:

#if defined (HAVE_SETLOCALE)
    setlocale (LC_ALL, "");
#endif

and it seems to work. The opengroup defines LC_ALL as an extension of
the ISO C standard:

http://www.opengroup.org/onlinepubs/009695399/functions/setlocale.html

and it also says "Application writers may make use of an extension as
it is supported on all IEEE Std 1003.1-2001-conforming systems".

Thanks.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]