ngettext support

binutils has lacked proper pluralization of output messages for a long
time, for example, readelf will display information about a section
that "contains 1 entries" or "There are 1 section headers".  Fixing
this properly requires us to use ngettext, because other languages
have different rules to English.

This patch defines macros for ngettext and friends to handle builds
with --disable-nls, and tidies the existing nls support.  I've
redefined gettext rather than just defining "_" as dgettext in bfd and
opcodes in case someone wants to use gettext there (which might
conceivably happen with generated code).

bfd/
	* sysdep.h: Formatting, comment fixes.
	(gettext, ngettext): Redefine when ENABLE_NLS.
	(ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
	(_): Define using gettext.
	(textdomain, bindtextdomain): Use safer "do nothing".
	* hosts/alphavms.h (textdomain, bindtextdomain): Likewise.
	(ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
opcodes/
	* opintl.h: Formatting, comment fixes.
	(gettext, ngettext): Redefine when ENABLE_NLS.
	(ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
	(_): Define using gettext.
	(textdomain, bindtextdomain): Use safer "do nothing".
binutils/
	* sysdep.h (textdomain, bindtextdomain): Use safer "do nothing".
	(ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
gas/
	* asintl.h (textdomain, bindtextdomain): Use safer "do nothing".
	(ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
gold/
	* system.h (textdomain, bindtextdomain): Use safer "do nothing".
	(ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
ld/
	* ld.h (textdomain, bindtextdomain): Use safer "do nothing".
	(ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 2362ca0a4fc8788c0fd40f5096ae5b8c8a9442b3..705179ac26f9033a3ba6d2476134c874617cff56 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,13 @@
+2017-11-07  Alan Modra  <amodra@gmail.com>
+
+       * sysdep.h: Formatting, comment fixes.
+       (gettext, ngettext): Redefine when ENABLE_NLS.
+       (ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
+       (_): Define using gettext.
+       (textdomain, bindtextdomain): Use safer "do nothing".
+       * hosts/alphavms.h (textdomain, bindtextdomain): Likewise.
+       (ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
+
 2017-11-05  Alan Modra  <amodra@gmail.com>
 
        PR 22397

diff --git a/bfd/hosts/alphavms.h b/bfd/hosts/alphavms.h
index 3acdd51a17e3879714d9b206f980300984cfe636..9e61c45bf039ed89d6e09fd6e6136567034f65ca 100644 (file)
--- a/bfd/hosts/alphavms.h
+++ b/bfd/hosts/alphavms.h
@@ -56,7 +56,13 @@ extern char *stpcpy (char *, const char *);
 #define gettext(Msgid) (Msgid)
 #define dgettext(Domainname, Msgid) (Msgid)
 #define dcgettext(Domainname, Msgid, Category) (Msgid)
-#define textdomain(Domainname) while (0) /* nothing */
-#define bindtextdomain(Domainname, Dirname) while (0) /* nothing */
+#define ngettext(Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+#define dngettext(Domainname, Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+#define dcngettext(Domainname, Msgid1, Msgid2, n, Category) \
+  (n == 1 ? Msgid1 : Msgid2)
+#define textdomain(Domainname) do {} while (0)
+#define bindtextdomain(Domainname, Dirname) do {} while (0)
 #define _(String) (String)
 #define N_(String) (String)

diff --git a/bfd/sysdep.h b/bfd/sysdep.h
index 6906da90c9892bd5267daeae052afd80d106470b..596850a83a5e81aa3b4244ae97d78de71b562ab6 100644 (file)
--- a/bfd/sysdep.h
+++ b/bfd/sysdep.h
@@ -181,31 +181,43 @@ size_t strnlen (const char *, size_t);
 #endif
 
 #ifdef ENABLE_NLS
-#include <libintl.h>
-/* Note the use of dgetext() and PACKAGE here, rather than gettext().
-
-   This is because the code in this directory is used to build a library which
-   will be linked with code in other directories to form programs.  We want to
-   maintain a seperate translation file for this directory however, rather
-   than being forced to merge it with that of any program linked to libbfd.
-   This is a library, so it cannot depend on the catalog currently loaded.
-
-   In order to do this, we have to make sure that when we extract messages we
-   use the OPCODES domain rather than the domain of the program that included
-   the bfd library, (eg OBJDUMP).  Hence we use dgettext (PACKAGE, String)
-   and define PACKAGE to be 'bfd'.  (See the code in configure).  */
-#define _(String) dgettext (PACKAGE, String)
-#ifdef gettext_noop
-#define N_(String) gettext_noop (String)
-#else
-#define N_(String) (String)
-#endif
+# include <libintl.h>
+/* Note the redefinition of gettext and ngettext here to use PACKAGE.
+
+   This is because the code in this directory is used to build a
+   library which will be linked with code in other directories to form
+   programs.  We want to maintain a seperate translation file for this
+   directory however, rather than being forced to merge it with that
+   of any program linked to libbfd.  This is a library, so it cannot
+   depend on the catalog currently loaded.
+
+   In order to do this, we have to make sure that when we extract
+   messages we use the BFD domain rather than the domain of the
+   program that included the bfd library, (eg OBJDUMP).  Hence we use
+   dgettext (PACKAGE, String) and define PACKAGE to be 'bfd'.
+   (See the code in configure).  */
+# undef gettext
+# define gettext(Msgid) dgettext (PACKAGE, Msgid)
+# undef ngettext
+# define ngettext(Msgid1, Msgid2, n) dngettext (PACKAGE, Msgid1, Msgid2, n)
+# define _(String) gettext (String)
+# ifdef gettext_noop
+#  define N_(String) gettext_noop (String)
+# else
+#  define N_(String) (String)
+# endif
 #else
 # define gettext(Msgid) (Msgid)
 # define dgettext(Domainname, Msgid) (Msgid)
 # define dcgettext(Domainname, Msgid, Category) (Msgid)
-# define textdomain(Domainname) while (0) /* nothing */
-# define bindtextdomain(Domainname, Dirname) while (0) /* nothing */
+# define ngettext(Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dngettext(Domainname, Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dcngettext(Domainname, Msgid1, Msgid2, n, Category) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define textdomain(Domainname) do {} while (0)
+# define bindtextdomain(Domainname, Dirname) do {} while (0)
 # define _(String) (String)
 # define N_(String) (String)
 #endif

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index ede61022f615313cf275afe8f5c6dc3b9a49fc71..caf17a9ac885bd97438222da01bce12e1d198d03 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,8 @@
+2017-11-07  Alan Modra  <amodra@gmail.com>
+
+       * sysdep.h (textdomain, bindtextdomain): Use safer "do nothing".
+       (ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
+
 2017-11-03  Claudiu Zissulescu  <claziss@synopsys.com>
 
        * doc/binutils.texi (ARC): Update disassembler options.

diff --git a/binutils/sysdep.h b/binutils/sysdep.h
index 3a89f9245240531870c2144245c17f2b5394cc3b..72ad8b7fdd9085785b091f3c059aa3b60d7172ad 100644 (file)
--- a/binutils/sysdep.h
+++ b/binutils/sysdep.h
@@ -159,8 +159,14 @@ size_t strnlen (const char *, size_t);
 # define gettext(Msgid) (Msgid)
 # define dgettext(Domainname, Msgid) (Msgid)
 # define dcgettext(Domainname, Msgid, Category) (Msgid)
-# define textdomain(Domainname) while (0) /* nothing */
-# define bindtextdomain(Domainname, Dirname) while (0) /* nothing */
+# define ngettext(Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dngettext(Domainname, Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dcngettext(Domainname, Msgid1, Msgid2, n, Category) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define textdomain(Domainname) do {} while (0)
+# define bindtextdomain(Domainname, Dirname) do {} while (0)
 # define _(String) (String)
 # define N_(String) (String)
 #endif

diff --git a/gas/ChangeLog b/gas/ChangeLog
index b48302d2f7c11bd5cf899de81e5b8c54c2424ca9..7ad6938d921bef746699995dd6e05e08954aa428 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2017-11-07  Alan Modra  <amodra@gmail.com>
+
+       * asintl.h (textdomain, bindtextdomain): Use safer "do nothing".
+       (ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
+
 2017-11-03  Siddhesh Poyarekar  <siddhesh.poyarekar@linaro.org>
            Jim Wilson  <jim.wilson@linaro.org>
 

diff --git a/gas/asintl.h b/gas/asintl.h
index 151ac3704820356cdede516df74cc2501a7a800f..e80c5d28b4bdc10876fa8d86162bd0cde4067683 100644 (file)
--- a/gas/asintl.h
+++ b/gas/asintl.h
@@ -45,8 +45,14 @@
 # define gettext(Msgid) (Msgid)
 # define dgettext(Domainname, Msgid) (Msgid)
 # define dcgettext(Domainname, Msgid, Category) (Msgid)
-# define textdomain(Domainname) while (0) /* nothing */
-# define bindtextdomain(Domainname, Dirname) while (0) /* nothing */
+# define ngettext(Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dngettext(Domainname, Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dcngettext(Domainname, Msgid1, Msgid2, n, Category) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define textdomain(Domainname) do {} while (0)
+# define bindtextdomain(Domainname, Dirname) do {} while (0)
 # define _(String) (String)
 # define N_(String) (String)
 #endif

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 4ab533d5775f8b5a536b6a43906238620adc67dc..2d0ae45dc824fdae6a85c6374ab527408a4d3ac6 100644 (file)
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,8 @@
+2017-11-07  Alan Modra  <amodra@gmail.com>
+
+       * system.h (textdomain, bindtextdomain): Use safer "do nothing".
+       (ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
+
 2017-10-25  Alan Modra  <amodra@gmail.com>
 
        * symtab.cc (Symbol_table::add_from_relobj): Match "__gnu_lto_slim"

diff --git a/gold/system.h b/gold/system.h
index 5a3e001f001ac95a32d70eb56f308d57e1f1591c..1c21ee1170ea2b521f52de4db7699b1d2706e48e 100644 (file)
--- a/gold/system.h
+++ b/gold/system.h
@@ -49,8 +49,14 @@
 # define gettext(Msgid) (Msgid)
 # define dgettext(Domainname, Msgid) (Msgid)
 # define dcgettext(Domainname, Msgid, Category) (Msgid)
-# define textdomain(Domainname) do {} while (0) /* nothing */
-# define bindtextdomain(Domainname, Dirname) do {} while (0) /* nothing */
+# define ngettext(Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dngettext(Domainname, Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dcngettext(Domainname, Msgid1, Msgid2, n, Category) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define textdomain(Domainname) do {} while (0)
+# define bindtextdomain(Domainname, Dirname) do {} while (0)
 # define _(String) (String)
 # define N_(String) (String)
 #endif

diff --git a/ld/ChangeLog b/ld/ChangeLog
index bdc98d6ecd07cbcd4d688125fe1f9590edc79c56..48d771fee88e4ce09be418b96cb5df89404315f2 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,8 @@
+2017-11-07  Alan Modra  <amodra@gmail.com>
+
+       * ld.h (textdomain, bindtextdomain): Use safer "do nothing".
+       (ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
+
 2017-11-01  Alan Modra  <amodra@gmail.com>
 
        * testsuite/ld-powerpc/ambiguousv2.d: Remove FIXME.

diff --git a/ld/ld.h b/ld/ld.h
index c6fa1247f00ef024426e4a8b40c8dd6962e8da86..26e7f89a28d84adce7187ca7e947a6f097693cb7 100644 (file)
--- a/ld/ld.h
+++ b/ld/ld.h
@@ -55,8 +55,14 @@
 # define gettext(Msgid) (Msgid)
 # define dgettext(Domainname, Msgid) (Msgid)
 # define dcgettext(Domainname, Msgid, Category) (Msgid)
-# define textdomain(Domainname) while (0) /* nothing */
-# define bindtextdomain(Domainname, Dirname) while (0) /* nothing */
+# define ngettext(Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dngettext(Domainname, Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dcngettext(Domainname, Msgid1, Msgid2, n, Category) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define textdomain(Domainname) do {} while (0)
+# define bindtextdomain(Domainname, Dirname) do {} while (0)
 # define _(String) (String)
 # define N_(String) (String)
 #endif

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index ecf548d0baf5c62e43b0008e74d9d5bc46785257..7107ee69a0d5bbc4b42b6d6254aeb5fb49b79112 100644 (file)
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,11 @@
+2017-11-07  Alan Modra  <amodra@gmail.com>
+
+       * opintl.h: Formatting, comment fixes.
+       (gettext, ngettext): Redefine when ENABLE_NLS.
+       (ngettext, dngettext, dcngettext): Define when !ENABLE_NLS.
+       (_): Define using gettext.
+       (textdomain, bindtextdomain): Use safer "do nothing".
+
 2017-11-03  Claudiu Zissulescu  <claziss@synopsys.com>
 
        * arc-dis.c (print_hex): New variable.

diff --git a/opcodes/opintl.h b/opcodes/opintl.h
index 18d05481d06cfd6a46c7592cb8f830510461dba8..0db8dd909dd20621335262c5916a8673dcf58edd 100644 (file)
--- a/opcodes/opintl.h
+++ b/opcodes/opintl.h
@@ -22,20 +22,25 @@
 
 #ifdef ENABLE_NLS
 # include <libintl.h>
-/* Note the use of dgetext() and PACKAGE here, rather than gettext().
-
-   This is because the code in this directory is used to build a library which
-   will be linked with code in other directories to form programs.  We want to
-   maintain a seperate translation file for this directory however, rather
-   than being forced to merge it with that of any program linked to
-   libopcodes.  This is a library, so it cannot depend on the catalog
-   currently loaded.
-
-   In order to do this, we have to make sure that when we extract messages we
-   use the OPCODES domain rather than the domain of the program that included
-   the opcodes library, (eg OBJDUMP).  Hence we use dgettext (PACKAGE, String)
-   and define PACKAGE to be 'opcodes'.  (See the code in configure).  */
-# define _(String) dgettext (PACKAGE, String)
+/* Note the redefinition of gettext and ngettext here to use PACKAGE.
+
+   This is because the code in this directory is used to build a
+   library which will be linked with code in other directories to form
+   programs.  We want to maintain a seperate translation file for this
+   directory however, rather than being forced to merge it with that
+   of any program linked to libopcodes.  This is a library, so it
+   cannot depend on the catalog currently loaded.
+
+   In order to do this, we have to make sure that when we extract
+   messages we use the OPCODES domain rather than the domain of the
+   program that included the opcodes library, (eg OBJDUMP).  Hence we
+   use dgettext (PACKAGE, String) and define PACKAGE to be 'opcodes'.
+   (See the code in configure).  */
+# undef gettext
+# define gettext(Msgid) dgettext (PACKAGE, Msgid)
+# undef ngettext
+# define ngettext(Msgid1, Msgid2, n) dngettext (PACKAGE, Msgid1, Msgid2, n)
+# define _(String) gettext (String)
 # ifdef gettext_noop
 #  define N_(String) gettext_noop (String)
 # else
@@ -45,8 +50,14 @@
 # define gettext(Msgid) (Msgid)
 # define dgettext(Domainname, Msgid) (Msgid)
 # define dcgettext(Domainname, Msgid, Category) (Msgid)
-# define textdomain(Domainname) while (0) /* nothing */
-# define bindtextdomain(Domainname, Dirname) while (0) /* nothing */
+# define ngettext(Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dngettext(Domainname, Msgid1, Msgid2, n) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define dcngettext(Domainname, Msgid1, Msgid2, n, Category) \
+  (n == 1 ? Msgid1 : Msgid2)
+# define textdomain(Domainname) do {} while (0)
+# define bindtextdomain(Domainname, Dirname) do {} while (0)
 # define _(String) (String)
 # define N_(String) (String)
 #endif