From aeb1b5f5b72081d190c9c24b3725cc09ce2bad1e Mon Sep 17 00:00:00 2001 From: Luc Van Oostenryck Date: Sun, 5 Jul 2020 01:40:33 +0200 Subject: sindex: avoid a warning with 'case -1:' When parsing the format, there is a 'case -1:' which: * seems to be there only for the following label (but mixing labels with cases, like here, is OK). * on architectures where chars are unsigned, the compiler complains: warning: case label value is less than minimum value for type So, remove the unneeded 'case -1:' and use an explicit 'default:' to catch invalid formats. Also, align the label with the cases, it looks nicer so. Signed-off-by: Luc Van Oostenryck Reviewed-by: Alexey Gladkov --- sindex.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/sindex.c b/sindex.c index ea092a4a..22836a95 100644 --- a/sindex.c +++ b/sindex.c @@ -970,8 +970,8 @@ static int search_query_callback(void *data, int argc, char **argv, char **colna print_file_line(argv[0], atoi(argv[1])); fmt++; break; - case -1: -print_string: + + print_string: if (n) { printf("%.*s", n, buf); n = 0; @@ -979,6 +979,9 @@ print_string: printf("%s", argv[colnum]); fmt++; break; + default: + break; + } if (pos == fmt) -- cgit 1.2.3-korg