From 9c46c7facda384f8fb6078e70b4e84ff84ed949d Mon Sep 17 00:00:00 2001 From: Mauro Carvalho Chehab Date: Fri, 25 Jul 2014 19:33:40 -0300 Subject: [PATCH] [media] radio-miropcm20: fix a compilation warning drivers/media/radio/radio-miropcm20.c: In function 'sanitize': drivers/media/radio/radio-miropcm20.c:216:3: warning: comparison is always false due to limited range of data type [-Wtype-limits] if (p[i] < 32 || p[i] >= 128) { ^ As p is declared as a char array, it is signed. So, it can never be bigger than 127. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/radio/radio-miropcm20.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/radio/radio-miropcm20.c b/drivers/media/radio/radio-miropcm20.c index 72df00e..ac9915d 100644 --- a/drivers/media/radio/radio-miropcm20.c +++ b/drivers/media/radio/radio-miropcm20.c @@ -213,7 +213,7 @@ static bool sanitize(char *p, int size) bool ret = true; for (i = 0; i < size; i++) { - if (p[i] < 32 || p[i] >= 128) { + if (p[i] < 32) { p[i] = ' '; ret = false; } -- 2.7.4