From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 27 Dec 2007 17:00:56 +0000 (+0000)
Subject: Fix ill-advised usage of x?y:z expressions in errmsg() and errhint() calls.
X-Git-Url: http://waps.l3s.uni-hannover.de/gitweb/?a=commitdiff_plain;h=73edcaa9136e760b3ccf2457cc9114cb88d10adf;p=users%2Fbernd%2Fpostgres.git

Fix ill-advised usage of x?y:z expressions in errmsg() and errhint() calls.
This prevented gettext from recognizing the strings that need to be
translated.
---

diff --git a/src/backend/commands/copy.c b/src/backend/commands/copy.c
index 5f3df3f99e..6bb706c65e 100644
--- a/src/backend/commands/copy.c
+++ b/src/backend/commands/copy.c
@@ -2420,12 +2420,12 @@ CopyReadLineText(CopyState cstate)
 					if (cstate->eol_type == EOL_CRNL)
 						ereport(ERROR,
 								(errcode(ERRCODE_BAD_COPY_FILE_FORMAT),
-								 errmsg(!cstate->csv_mode ?
-									"literal carriage return found in data" :
-								   "unquoted carriage return found in data"),
-								 errhint(!cstate->csv_mode ?
-								"Use \"\\r\" to represent carriage return." :
-										 "Use quoted CSV field to represent carriage return.")));
+								 !cstate->csv_mode ?
+								 errmsg("literal carriage return found in data") :
+								 errmsg("unquoted carriage return found in data"),
+								 !cstate->csv_mode ?
+								 errhint("Use \"\\r\" to represent carriage return.") :
+								 errhint("Use quoted CSV field to represent carriage return.")));
 
 					/*
 					 * if we got here, it is the first line and we didn't find
@@ -2437,12 +2437,12 @@ CopyReadLineText(CopyState cstate)
 			else if (cstate->eol_type == EOL_NL)
 				ereport(ERROR,
 						(errcode(ERRCODE_BAD_COPY_FILE_FORMAT),
-						 errmsg(!cstate->csv_mode ?
-								"literal carriage return found in data" :
-								"unquoted carriage return found in data"),
-						 errhint(!cstate->csv_mode ?
-								 "Use \"\\r\" to represent carriage return." :
-					 "Use quoted CSV field to represent carriage return.")));
+						 !cstate->csv_mode ?
+						 errmsg("literal carriage return found in data") :
+						 errmsg("unquoted carriage return found in data"),
+						 !cstate->csv_mode ?
+						 errhint("Use \"\\r\" to represent carriage return.") :
+						 errhint("Use quoted CSV field to represent carriage return.")));
 			/* If reach here, we have found the line terminator */
 			break;
 		}
@@ -2453,12 +2453,12 @@ CopyReadLineText(CopyState cstate)
 			if (cstate->eol_type == EOL_CR || cstate->eol_type == EOL_CRNL)
 				ereport(ERROR,
 						(errcode(ERRCODE_BAD_COPY_FILE_FORMAT),
-						 errmsg(!cstate->csv_mode ?
-								"literal newline found in data" :
-								"unquoted newline found in data"),
-						 errhint(!cstate->csv_mode ?
-								 "Use \"\\n\" to represent newline." :
-							 "Use quoted CSV field to represent newline.")));
+						 !cstate->csv_mode ?
+						 errmsg("literal newline found in data") :
+						 errmsg("unquoted newline found in data"),
+						 !cstate->csv_mode ?
+						 errhint("Use \"\\n\" to represent newline.") :
+						 errhint("Use quoted CSV field to represent newline.")));
 			cstate->eol_type = EOL_NL;	/* in case not set yet */
 			/* If reach here, we have found the line terminator */
 			break;