Make GUC_check_errdetail messages full sentences

They were all missing punctuation, one was missing initial capital.
Per our message style guidelines.

No backpatch, to avoid breaking existing translations.
pull/185/head
Álvaro Herrera 10 months ago
parent fd9924542b
commit 6ba9892f5c
No known key found for this signature in database
GPG Key ID: 1C20ACB9D5C564AE
  1. 2
      src/backend/access/transam/slru.c
  2. 2
      src/backend/access/transam/xlogrecovery.c
  3. 3
      src/backend/commands/vacuum.c
  4. 4
      src/backend/replication/slot.c
  5. 3
      src/backend/replication/syncrep.c
  6. 8
      src/backend/storage/file/fd.c
  7. 2
      src/backend/utils/error/elog.c

@ -358,7 +358,7 @@ check_slru_buffers(const char *name, int *newval)
if (*newval % SLRU_BANK_SIZE == 0) if (*newval % SLRU_BANK_SIZE == 0)
return true; return true;
GUC_check_errdetail("\"%s\" must be a multiple of %d", name, GUC_check_errdetail("\"%s\" must be a multiple of %d.", name,
SLRU_BANK_SIZE); SLRU_BANK_SIZE);
return false; return false;
} }

@ -4933,7 +4933,7 @@ check_recovery_target_time(char **newval, void **extra, GucSource source)
if (tm2timestamp(tm, fsec, &tz, &timestamp) != 0) if (tm2timestamp(tm, fsec, &tz, &timestamp) != 0)
{ {
GUC_check_errdetail("timestamp out of range: \"%s\"", str); GUC_check_errdetail("Timestamp out of range: \"%s\".", str);
return false; return false;
} }
} }

@ -130,7 +130,8 @@ check_vacuum_buffer_usage_limit(int *newval, void **extra,
return true; return true;
/* Value does not fall within any allowable range */ /* Value does not fall within any allowable range */
GUC_check_errdetail("\"vacuum_buffer_usage_limit\" must be 0 or between %d kB and %d kB", GUC_check_errdetail("\"%s\" must be 0 or between %d kB and %d kB.",
"vacuum_buffer_usage_limit",
MIN_BAS_VAC_RING_SIZE_KB, MAX_BAS_VAC_RING_SIZE_KB); MIN_BAS_VAC_RING_SIZE_KB, MAX_BAS_VAC_RING_SIZE_KB);
return false; return false;

@ -2478,7 +2478,7 @@ validate_sync_standby_slots(char *rawname, List **elemlist)
if (!slot) if (!slot)
{ {
GUC_check_errdetail("replication slot \"%s\" does not exist", GUC_check_errdetail("Replication slot \"%s\" does not exist.",
name); name);
ok = false; ok = false;
break; break;
@ -2486,7 +2486,7 @@ validate_sync_standby_slots(char *rawname, List **elemlist)
if (!SlotIsPhysical(slot)) if (!SlotIsPhysical(slot))
{ {
GUC_check_errdetail("\"%s\" is not a physical replication slot", GUC_check_errdetail("\"%s\" is not a physical replication slot.",
name); name);
ok = false; ok = false;
break; break;

@ -1010,7 +1010,8 @@ check_synchronous_standby_names(char **newval, void **extra, GucSource source)
if (syncrep_parse_error_msg) if (syncrep_parse_error_msg)
GUC_check_errdetail("%s", syncrep_parse_error_msg); GUC_check_errdetail("%s", syncrep_parse_error_msg);
else else
GUC_check_errdetail("\"synchronous_standby_names\" parser failed"); GUC_check_errdetail("\"%s\" parser failed.",
"synchronous_standby_names");
return false; return false;
} }

@ -3985,7 +3985,7 @@ check_debug_io_direct(char **newval, void **extra, GucSource source)
if (!SplitGUCList(rawstring, ',', &elemlist)) if (!SplitGUCList(rawstring, ',', &elemlist))
{ {
GUC_check_errdetail("Invalid list syntax in parameter \"%s\"", GUC_check_errdetail("Invalid list syntax in parameter \"%s\".",
"debug_io_direct"); "debug_io_direct");
pfree(rawstring); pfree(rawstring);
list_free(elemlist); list_free(elemlist);
@ -4005,7 +4005,7 @@ check_debug_io_direct(char **newval, void **extra, GucSource source)
flags |= IO_DIRECT_WAL_INIT; flags |= IO_DIRECT_WAL_INIT;
else else
{ {
GUC_check_errdetail("Invalid option \"%s\"", item); GUC_check_errdetail("Invalid option \"%s\".", item);
result = false; result = false;
break; break;
} }
@ -4018,7 +4018,7 @@ check_debug_io_direct(char **newval, void **extra, GucSource source)
#if XLOG_BLCKSZ < PG_IO_ALIGN_SIZE #if XLOG_BLCKSZ < PG_IO_ALIGN_SIZE
if (result && (flags & (IO_DIRECT_WAL | IO_DIRECT_WAL_INIT))) if (result && (flags & (IO_DIRECT_WAL | IO_DIRECT_WAL_INIT)))
{ {
GUC_check_errdetail("\"%s\" is not supported for WAL because %s is too small", GUC_check_errdetail("\"%s\" is not supported for WAL because %s is too small.",
"debug_io_direct", "XLOG_BLCKSZ"); "debug_io_direct", "XLOG_BLCKSZ");
result = false; result = false;
} }
@ -4026,7 +4026,7 @@ check_debug_io_direct(char **newval, void **extra, GucSource source)
#if BLCKSZ < PG_IO_ALIGN_SIZE #if BLCKSZ < PG_IO_ALIGN_SIZE
if (result && (flags & IO_DIRECT_DATA)) if (result && (flags & IO_DIRECT_DATA))
{ {
GUC_check_errdetail("\"%s\" is not supported for WAL because %s is too small", GUC_check_errdetail("\"%s\" is not supported for WAL because %s is too small.",
"debug_io_direct", "BLCKSZ"); "debug_io_direct", "BLCKSZ");
result = false; result = false;
} }

@ -2183,7 +2183,7 @@ check_backtrace_functions(char **newval, void **extra, GucSource source)
", \n\t"); ", \n\t");
if (validlen != newvallen) if (validlen != newvallen)
{ {
GUC_check_errdetail("Invalid character"); GUC_check_errdetail("Invalid character.");
return false; return false;
} }

Loading…
Cancel
Save