summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhilip Withnall <withnall@endlessm.com>2019-11-14 14:16:13 +0000
committerPhilip Withnall <withnall@endlessm.com>2019-11-19 15:43:17 +0000
commite935a1b0c300d8fc97661e01a8200af14876e627 (patch)
treed9aa9f4b0cfde9c84775d9485aae77ea8d61f374
parentd35abf7b17501ec726d309d14a58696ac07126bf (diff)
downloaddesktop-file-utils-e935a1b0c300d8fc97661e01a8200af14876e627.tar.gz
desktop-file-validate: Simplify a couple of error messages
They were printing the value of both keys, but given that the warning is that the two values are (almost) the same, it seems a pointless waste of space to print both of them. Print only one. Signed-off-by: Philip Withnall <withnall@endlessm.com>
-rw-r--r--src/validate.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/validate.c b/src/validate.c
index 39f8ddc..f98bcf1 100644
--- a/src/validate.c
+++ b/src/validate.c
@@ -993,9 +993,9 @@ handle_comment_key (kf_validator *kf,
if (keyvalue && g_ascii_strcasecmp (value, keyvalue->value) == 0) {
print_warning (kf, "value \"%s\" for key \"%s\" in group \"%s\" "
- "looks redundant with value \"%s\" of key \"%s\"\n",
+ "looks the same as that of key \"%s\"\n",
value, locale_key, kf->current_group,
- keyvalue->value, keyvalue->key);
+ keyvalue->key);
return FALSE;
}
@@ -1006,9 +1006,9 @@ handle_comment_key (kf_validator *kf,
if (keyvalue && g_ascii_strcasecmp (value, keyvalue->value) == 0) {
print_warning (kf, "value \"%s\" for key \"%s\" in group \"%s\" "
- "looks redundant with value \"%s\" of key \"%s\"\n",
+ "looks the same as that of key \"%s\"\n",
value, locale_key, kf->current_group,
- keyvalue->value, keyvalue->key);
+ keyvalue->key);
return FALSE;
}