diff options
author | Philip Withnall <withnall@endlessm.com> | 2019-11-14 14:16:13 +0000 |
---|---|---|
committer | Philip Withnall <withnall@endlessm.com> | 2019-11-19 15:43:17 +0000 |
commit | e935a1b0c300d8fc97661e01a8200af14876e627 (patch) | |
tree | d9aa9f4b0cfde9c84775d9485aae77ea8d61f374 | |
parent | d35abf7b17501ec726d309d14a58696ac07126bf (diff) | |
download | desktop-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.c | 8 |
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; } |