Fix NLS for incorrect GUC enum value hint message

The translation markers were applied at the wrong place, so no string was extracted for translation. Also add translator comments here and in a similar place. Reviewed-by: Álvaro Herrera <alvherre@kurilemu.de> Discussion: https://www.postgresql.org/message-id/2c961fa1-14f6-44a2-985c-e30b95654e8d%40eisentraut.org

Author: petere

src/backend/replication/logical/relation.c

@@ -249,6 +249,7 @@ logicalrep_get_attrs_str(LogicalRepRelation *remoterel, Bitmapset *atts)
 {
 attcnt++;
 if (attcnt > 1)
+/* translator: This is a separator in a list of entity names. */
 appendStringInfoString(&attsbuf, _(", "));
 
 appendStringInfo(&attsbuf, _("\"%s\""), remoterel->attnames[i]);

src/backend/utils/misc/guc.c

@@ -3135,14 +3135,17 @@ parse_and_validate_value(const struct config_generic *record,
 char *hintmsg;
 
 hintmsg = config_enum_get_options(conf,
- "Available values: ",
- ".", ", ");
+ _("Available values: "),
+ /* translator: This is the terminator of a list of entity names. */
+ _("."),
+ /* translator: This is a separator in a list of entity names. */
+ _(", "));
 
 ereport(elevel,
 (errcode(ERRCODE_INVALID_PARAMETER_VALUE),
  errmsg("invalid value for parameter \"%s\": \"%s\"",
 record->name, value),
- hintmsg ? errhint("%s", _(hintmsg)) : 0));
+ hintmsg ? errhint("%s", hintmsg) : 0));
 
 if (hintmsg)
 pfree(hintmsg);