From 73a5261929410f2a772d63e5b034af5aa9435531 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Wed, 31 May 2000 18:32:07 +0000 Subject: [PATCH] Add comment describoing why dgettext() is used in _() macro. --- bfd/ChangeLog | 5 +++++ bfd/sysdep.h | 12 ++++++++++++ opcodes/ChangeLog | 5 +++++ opcodes/opintl.h | 13 +++++++++++++ 4 files changed, 35 insertions(+) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 03fc856ff3..798ed4d58d 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2000-05-31 Nick Clifton + + * opintl.h (_(String)): Explain why dgettext is used instead + of gettext. + 2000-05-31 Ulrich Drepper * elf.c (_bfd_elf_slurp_version_tables): Correct reading of version diff --git a/bfd/sysdep.h b/bfd/sysdep.h index 77b5e50a9c..bf3eac03cc 100644 --- a/bfd/sysdep.h +++ b/bfd/sysdep.h @@ -126,6 +126,18 @@ extern char *getenv (); #ifdef ENABLE_NLS #include +/* 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) diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog index 08b2c16749..a2c890f86a 100644 --- a/opcodes/ChangeLog +++ b/opcodes/ChangeLog @@ -1,3 +1,8 @@ +2000-05-31 Nick Clifton + + * opintl.h (_(String)): Explain why dgettext is used instead of + gettext. + 2000-05-30 Nick Clifton * opintl.h (gettext, dgettext, dcgettext, textdomain, diff --git a/opcodes/opintl.h b/opcodes/opintl.h index 55a989a53b..492fc1a6c8 100644 --- a/opcodes/opintl.h +++ b/opcodes/opintl.h @@ -12,6 +12,19 @@ #ifdef ENABLE_NLS # include +/* 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) # ifdef gettext_noop # define N_(String) gettext_noop (String) -- 2.11.0