missing translation marker

Hi,

It seems there's a missing gettext() call in new code in hba.c, per the
patch below. I don't actually like this, but it seems the alternative
would be to create a variadic function which I believe to be
overengineering ...

Index: hba.c
===================================================================
RCS file: /home/alvherre/Code/cvs/pgsql/src/backend/libpq/hba.c,v
retrieving revision 1.184
diff -c -p -u -r1.184 hba.c
cvs diff: conflicting specifications of output style
--- hba.c	25 Mar 2009 14:12:02 -0000	1.184
+++ hba.c	15 Apr 2009 16:50:21 -0000
@@ -579,7 +579,7 @@ check_db(const char *dbname, const char 
 	ereport(LOG, \
 			(errcode(ERRCODE_CONFIG_FILE_ERROR), \
 			 errmsg("authentication option \"%s\" is only valid for authentication methods \"%s\"", \
-					optname, validmethods), \
+					optname, _(validmethods)), \
 			 errcontext("line %d of configuration file \"%s\"", \
 					line_num, HbaFileName))); \
 	return false; \
@@ -952,7 +952,7 @@ parse_hba_line(List *line, int line_num,
 					parsedline->auth_method != uaGSS &&
 					parsedline->auth_method != uaSSPI &&
 					parsedline->auth_method != uaCert)
-					INVALID_AUTH_OPTION("map", "ident, krb5, gssapi, sspi and cert");
+					INVALID_AUTH_OPTION("map", gettext_noop("ident, krb5, gssapi, sspi and cert"));
 				parsedline->usermap = pstrdup(c);
 			}
 			else if (strcmp(token, "clientcert") == 0)
@@ -1050,7 +1050,7 @@ parse_hba_line(List *line, int line_num,
 				if (parsedline->auth_method != uaKrb5 &&
 					parsedline->auth_method != uaGSS &&
 					parsedline->auth_method != uaSSPI)
-					INVALID_AUTH_OPTION("krb_realm", "krb5, gssapi and sspi");
+					INVALID_AUTH_OPTION("krb_realm", gettext_noop("krb5, gssapi and sspi"));
 				parsedline->krb_realm = pstrdup(c);
 			}
 			else if (strcmp(token, "include_realm") == 0)
@@ -1058,7 +1058,7 @@ parse_hba_line(List *line, int line_num,
 				if (parsedline->auth_method != uaKrb5 &&
 					parsedline->auth_method != uaGSS &&
 					parsedline->auth_method != uaSSPI)
-					INVALID_AUTH_OPTION("include_realm", "krb5, gssapi and sspi");
+					INVALID_AUTH_OPTION("include_realm", gettext_noop("krb5, gssapi and sspi"));
 				if (strcmp(c, "1") == 0)
 					parsedline->include_realm = true;
 				else

--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support