fs/adfs/adfs.h: fix unsigned comparison
authorAndrew Morton <akpm@linux-foundation.org>
Wed, 23 Mar 2011 23:41:43 +0000 (16:41 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 24 Mar 2011 02:46:09 +0000 (19:46 -0700)
fs/adfs/adfs.h: In function 'append_filetype_suffix':
fs/adfs/adfs.h:115: warning: comparison is always false due to limited range of data type

Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Stuart Swales <stuart.swales.croftnuisk@gmail.com>
Cc: Russell King <rmk+kernel@arm.linux.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/adfs/adfs.h

index a8a58d864f960d89cccd8e25c29f05826f2b290e..718ac1f440c6729c334b800375d853a5bd107366 100644 (file)
@@ -112,7 +112,7 @@ struct object_info {
 /* RISC OS 12-bit filetype converts to ,xyz hex filename suffix */
 static inline int append_filetype_suffix(char *buf, __u16 filetype)
 {
-       if (filetype == -1)
+       if (filetype == 0xffff) /* no explicit 12-bit file type was set */
                return 0;
 
        *buf++ = ',';