Messages in this thread Patch in this message | | | Date | Tue, 10 Apr 2007 07:32:12 -0400 (EDT) | From | "Robert P. J. Day" <> | Subject | [PATCH] VIDEO: Add spaces on either side of the case "..." operator. |
| |
Following the programming advice laid down in the gcc manual, make sure the case "..." operator has spaces on either side.
Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>
---
These are the only other examples of this in the entire kernel source tree, so no need to worry about an impending deluge of related patches.
drivers/video/modedb.c | 4 ++-- drivers/video/pxafb.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/video/modedb.c b/drivers/video/modedb.c index 3e51794..3741ad7 100644 --- a/drivers/video/modedb.c +++ b/drivers/video/modedb.c @@ -395,7 +395,7 @@ static int my_atoi(const char *name)
for (;; name++) { switch (*name) { - case '0'...'9': + case '0' ... '9': val = 10*val+(*name-'0'); break; default: @@ -548,7 +548,7 @@ int fb_find_mode(struct fb_var_screeninfo *var, } else goto done; break; - case '0'...'9': + case '0' ... '9': break; case 'M': if (!yres_specified) diff --git a/drivers/video/pxafb.c b/drivers/video/pxafb.c index b4947c8..4a6cacc 100644 --- a/drivers/video/pxafb.c +++ b/drivers/video/pxafb.c @@ -1203,7 +1203,7 @@ static int __init pxafb_parse_options(struct device *dev, char *options) } else goto done; break; - case '0'...'9': + case '0' ... '9': break; default: goto done; -- ======================================================================== Robert P. J. Day Linux Consulting, Training and Annoying Kernel Pedantry Waterloo, Ontario, CANADA http://fsdev.net/wiki/index.php?title=Main_Page ======================================================================== - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |