This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH] Remove erroneous marks for translation
- From: Philipp Thomas <pthomas at suse dot de>
- To: gcc-patches at gcc dot gnu dot org
- Cc: Jan Hubicka <jh at suse dot cz>
- Date: Sun, 17 Feb 2002 08:57:27 +0100
- Subject: [PATCH] Remove erroneous marks for translation
Marking empty strings for translations is useless and also erroneously as
gettext won't return an empty string but something else entirely.
Regression checked on i686-suse-linux and checked in under the obvious bug
rule.
BTW, can anybody explain me the reason for having *two* entries each for
no-3dnow, no-sse and no-sse2, one documented and the other not? At least to
me this looks fishy.
2002-02-17 Philipp Thomas <pthomas@suse.de>
* config/i386/i386.h: Don't mark empty strings for translation.
Index: i386.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/i386/i386.h,v
retrieving revision 1.242
diff -u -p -r1.242 i386.h
--- i386.h 2002/02/07 11:18:32 1.242
+++ i386.h 2002/02/17 07:48:53
@@ -281,18 +281,23 @@ extern int x86_prefetch_sse;
#define TARGET_RED_ZONE (!(target_flags & MASK_NO_RED_ZONE))
+/* WARNING: Do not mark empty strings for translation, as calling
+ gettext on an empty string does NOT return an empty
+ string. */
+
+
#define TARGET_SWITCHES \
{ { "80387", MASK_80387, N_("Use hardware fp") }, \
{ "no-80387", -MASK_80387, N_("Do not use hardware fp") }, \
{ "hard-float", MASK_80387, N_("Use hardware fp") }, \
{ "soft-float", -MASK_80387, N_("Do not use hardware fp") }, \
{ "no-soft-float", MASK_80387, N_("Use hardware fp") }, \
- { "386", 0, N_("") /*Deprecated.*/}, \
- { "486", 0, N_("") /*Deprecated.*/}, \
- { "pentium", 0, N_("") /*Deprecated.*/}, \
- { "pentiumpro", 0, N_("") /*Deprecated.*/}, \
- { "intel-syntax", 0, N_("") /*Deprecated.*/}, \
- { "no-intel-syntax", 0, N_("") /*Deprecated.*/}, \
+ { "386", 0, "" /*Deprecated.*/}, \
+ { "486", 0, "" /*Deprecated.*/}, \
+ { "pentium", 0, "" /*Deprecated.*/}, \
+ { "pentiumpro", 0, "" /*Deprecated.*/}, \
+ { "intel-syntax", 0, "" /*Deprecated.*/}, \
+ { "no-intel-syntax", 0, "" /*Deprecated.*/}, \
{ "rtd", MASK_RTD, \
N_("Alternate calling convention") }, \
{ "no-rtd", -MASK_RTD, \
@@ -346,20 +351,20 @@ extern int x86_prefetch_sse;
N_("Support MMX built-in functions") }, \
{ "no-mmx", -MASK_MMX, \
N_("Do not support MMX built-in functions") }, \
- { "no-mmx", MASK_MMX_SET, N_("") }, \
+ { "no-mmx", MASK_MMX_SET, "" }, \
{ "3dnow", MASK_3DNOW | MASK_3DNOW_SET, \
N_("Support 3DNow! built-in functions") }, \
- { "no-3dnow", -MASK_3DNOW, N_("") }, \
+ { "no-3dnow", -MASK_3DNOW, "" }, \
{ "no-3dnow", MASK_3DNOW_SET, \
N_("Do not support 3DNow! built-in functions") }, \
{ "sse", MASK_SSE | MASK_SSE_SET, \
N_("Support MMX and SSE built-in functions and code generation") }, \
- { "no-sse", -MASK_SSE, N_("") }, \
+ { "no-sse", -MASK_SSE, "" }, \
{ "no-sse", MASK_SSE_SET, \
N_("Do not support MMX and SSE built-in functions and code generation") },\
{ "sse2", MASK_SSE2 | MASK_SSE2_SET, \
N_("Support MMX, SSE and SSE2 built-in functions and code generation") }, \
- { "no-sse2", -MASK_SSE2, N_("") }, \
+ { "no-sse2", -MASK_SSE2, "" }, \
{ "no-sse2", MASK_SSE2_SET, \
N_("Do not support MMX, SSE and SSE2 built-in functions and code generation") }, \
{ "128bit-long-double", MASK_128BIT_LONG_DOUBLE, \
@@ -440,9 +445,9 @@ extern int ix86_arch;
{ "cmodel=", &ix86_cmodel_string, \
N_("Use given x86-64 code model") }, \
{ "debug-arg", &ix86_debug_arg_string, \
- N_("" /* Undocumented. */) }, \
+ "" /* Undocumented. */ }, \
{ "debug-addr", &ix86_debug_addr_string, \
- N_("" /* Undocumented. */) }, \
+ "" /* Undocumented. */ }, \
{ "asm=", &ix86_asm_string, \
N_("Use given assembler dialect") }, \
SUBTARGET_OPTIONS \
--
Philipp Thomas <pthomas@suse.de>
SuSE Linux AG, Deutscherrnstr. 15-19, D-90429 Nuremberg, Germany