* [PATCH 8/9] Fixed indentation violation in switch statement : Aligned the "switch" and its subordinate "case" labels in the same column instead of "double-intending" the "case" labels.
@ 2015-01-17 11:53 Shailendra Verma
2015-01-17 15:13 ` Antonio Ospite
0 siblings, 1 reply; 3+ messages in thread
From: Shailendra Verma @ 2015-01-17 11:53 UTC (permalink / raw)
To: Dmitry Torokhov, linux-input; +Cc: linux-kernel, Shailendra Verma
Signed-off-by: Shailendra Verma <shailendra.capricorn@gmail.com>
---
drivers/input/evdev.c | 30 ++++++++++++++++++++----------
drivers/input/mousedev.c | 18 ++++++++++++------
2 files changed, 32 insertions(+), 16 deletions(-)
diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c
index 18d4b2c..85ee594 100644
--- a/drivers/input/evdev.c
+++ b/drivers/input/evdev.c
@@ -667,16 +667,26 @@ static int handle_eviocgbit(struct input_dev *dev,
switch (type) {
- case 0: bits = dev->evbit; len = EV_MAX; break;
- case EV_KEY: bits = dev->keybit; len = KEY_MAX; break;
- case EV_REL: bits = dev->relbit; len = REL_MAX; break;
- case EV_ABS: bits = dev->absbit; len = ABS_MAX; break;
- case EV_MSC: bits = dev->mscbit; len = MSC_MAX; break;
- case EV_LED: bits = dev->ledbit; len = LED_MAX; break;
- case EV_SND: bits = dev->sndbit; len = SND_MAX; break;
- case EV_FF: bits = dev->ffbit; len = FF_MAX; break;
- case EV_SW: bits = dev->swbit; len = SW_MAX; break;
- default: return -EINVAL;
+ case 0:
+ bits = dev->evbit; len = EV_MAX; break;
+ case EV_KEY:
+ bits = dev->keybit; len = KEY_MAX; break;
+ case EV_REL:
+ bits = dev->relbit; len = REL_MAX; break;
+ case EV_ABS:
+ bits = dev->absbit; len = ABS_MAX; break;
+ case EV_MSC:
+ bits = dev->mscbit; len = MSC_MAX; break;
+ case EV_LED:
+ bits = dev->ledbit; len = LED_MAX; break;
+ case EV_SND:
+ bits = dev->sndbit; len = SND_MAX; break;
+ case EV_FF:
+ bits = dev->ffbit; len = FF_MAX; break;
+ case EV_SW:
+ bits = dev->swbit; len = SW_MAX; break;
+ default:
+ return -EINVAL;
}
return bits_to_user(bits, len, size, p, compat_mode);
diff --git a/drivers/input/mousedev.c b/drivers/input/mousedev.c
index b604564..a9857dd 100644
--- a/drivers/input/mousedev.c
+++ b/drivers/input/mousedev.c
@@ -230,25 +230,31 @@ static void mousedev_key_event(struct mousedev *mousedev,
case BTN_TOUCH:
case BTN_0:
- case BTN_LEFT: index = 0; break;
+ case BTN_LEFT:
+ index = 0; break;
case BTN_STYLUS:
case BTN_1:
- case BTN_RIGHT: index = 1; break;
+ case BTN_RIGHT:
+ index = 1; break;
case BTN_2:
case BTN_FORWARD:
case BTN_STYLUS2:
- case BTN_MIDDLE: index = 2; break;
+ case BTN_MIDDLE:
+ index = 2; break;
case BTN_3:
case BTN_BACK:
- case BTN_SIDE: index = 3; break;
+ case BTN_SIDE:
+ index = 3; break;
case BTN_4:
- case BTN_EXTRA: index = 4; break;
+ case BTN_EXTRA:
+ index = 4; break;
- default: return;
+ default:
+ return;
}
if (value) {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 8/9] Fixed indentation violation in switch statement : Aligned the "switch" and its subordinate "case" labels in the same column instead of "double-intending" the "case" labels.
2015-01-17 11:53 [PATCH 8/9] Fixed indentation violation in switch statement : Aligned the "switch" and its subordinate "case" labels in the same column instead of "double-intending" the "case" labels Shailendra Verma
@ 2015-01-17 15:13 ` Antonio Ospite
2015-01-17 22:15 ` Dmitry Torokhov
0 siblings, 1 reply; 3+ messages in thread
From: Antonio Ospite @ 2015-01-17 15:13 UTC (permalink / raw)
To: Shailendra Verma; +Cc: Dmitry Torokhov, linux-input, linux-kernel
On Sat, 17 Jan 2015 17:23:45 +0530
Shailendra Verma <shailendra.capricorn@gmail.com> wrote:
>
> Signed-off-by: Shailendra Verma <shailendra.capricorn@gmail.com>
> ---
> drivers/input/evdev.c | 30 ++++++++++++++++++++----------
> drivers/input/mousedev.c | 18 ++++++++++++------
> 2 files changed, 32 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c
> index 18d4b2c..85ee594 100644
> --- a/drivers/input/evdev.c
> +++ b/drivers/input/evdev.c
> @@ -667,16 +667,26 @@ static int handle_eviocgbit(struct input_dev *dev,
>
> switch (type) {
>
> - case 0: bits = dev->evbit; len = EV_MAX; break;
> - case EV_KEY: bits = dev->keybit; len = KEY_MAX; break;
> - case EV_REL: bits = dev->relbit; len = REL_MAX; break;
> - case EV_ABS: bits = dev->absbit; len = ABS_MAX; break;
> - case EV_MSC: bits = dev->mscbit; len = MSC_MAX; break;
> - case EV_LED: bits = dev->ledbit; len = LED_MAX; break;
> - case EV_SND: bits = dev->sndbit; len = SND_MAX; break;
> - case EV_FF: bits = dev->ffbit; len = FF_MAX; break;
> - case EV_SW: bits = dev->swbit; len = SW_MAX; break;
> - default: return -EINVAL;
I have the feeling this formatting was deliberate, to make comparing the
lines easier, see the alignment after colons and semicolons.
> + case 0:
> + bits = dev->evbit; len = EV_MAX; break;
> + case EV_KEY:
> + bits = dev->keybit; len = KEY_MAX; break;
> + case EV_REL:
> + bits = dev->relbit; len = REL_MAX; break;
> + case EV_ABS:
> + bits = dev->absbit; len = ABS_MAX; break;
> + case EV_MSC:
> + bits = dev->mscbit; len = MSC_MAX; break;
> + case EV_LED:
> + bits = dev->ledbit; len = LED_MAX; break;
> + case EV_SND:
> + bits = dev->sndbit; len = SND_MAX; break;
> + case EV_FF:
> + bits = dev->ffbit; len = FF_MAX; break;
> + case EV_SW:
> + bits = dev->swbit; len = SW_MAX; break;
> + default:
> + return -EINVAL;
> }
>
Now, this is a mixed style, multiple statements are still on the same
line which is also somewhat against the code style.
FWIW I'd leave the code current code untouched.
> return bits_to_user(bits, len, size, p, compat_mode);
> diff --git a/drivers/input/mousedev.c b/drivers/input/mousedev.c
> index b604564..a9857dd 100644
> --- a/drivers/input/mousedev.c
> +++ b/drivers/input/mousedev.c
> @@ -230,25 +230,31 @@ static void mousedev_key_event(struct mousedev *mousedev,
>
> case BTN_TOUCH:
> case BTN_0:
> - case BTN_LEFT: index = 0; break;
> + case BTN_LEFT:
> + index = 0; break;
>
This makes more sense indeed but the break is still on the same line of
the assignment, if you change the style fix it for good and put the
break on its own line.
> case BTN_STYLUS:
> case BTN_1:
> - case BTN_RIGHT: index = 1; break;
> + case BTN_RIGHT:
> + index = 1; break;
>
> case BTN_2:
> case BTN_FORWARD:
> case BTN_STYLUS2:
> - case BTN_MIDDLE: index = 2; break;
> + case BTN_MIDDLE:
> + index = 2; break;
>
> case BTN_3:
> case BTN_BACK:
> - case BTN_SIDE: index = 3; break;
> + case BTN_SIDE:
> + index = 3; break;
>
> case BTN_4:
> - case BTN_EXTRA: index = 4; break;
> + case BTN_EXTRA:
> + index = 4; break;
>
> - default: return;
> + default:
> + return;
> }
>
> if (value) {
> --
> 1.7.9.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-input" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Antonio Ospite
http://ao2.it
A: Because it messes up the order in which people normally read text.
See http://en.wikipedia.org/wiki/Posting_style
Q: Why is top-posting such a bad thing?
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 8/9] Fixed indentation violation in switch statement : Aligned the "switch" and its subordinate "case" labels in the same column instead of "double-intending" the "case" labels.
2015-01-17 15:13 ` Antonio Ospite
@ 2015-01-17 22:15 ` Dmitry Torokhov
0 siblings, 0 replies; 3+ messages in thread
From: Dmitry Torokhov @ 2015-01-17 22:15 UTC (permalink / raw)
To: Antonio Ospite; +Cc: Shailendra Verma, linux-input, linux-kernel
On Sat, Jan 17, 2015 at 04:13:16PM +0100, Antonio Ospite wrote:
> On Sat, 17 Jan 2015 17:23:45 +0530
> Shailendra Verma <shailendra.capricorn@gmail.com> wrote:
>
> >
> > Signed-off-by: Shailendra Verma <shailendra.capricorn@gmail.com>
> > ---
> > drivers/input/evdev.c | 30 ++++++++++++++++++++----------
> > drivers/input/mousedev.c | 18 ++++++++++++------
> > 2 files changed, 32 insertions(+), 16 deletions(-)
> >
> > diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c
> > index 18d4b2c..85ee594 100644
> > --- a/drivers/input/evdev.c
> > +++ b/drivers/input/evdev.c
> > @@ -667,16 +667,26 @@ static int handle_eviocgbit(struct input_dev *dev,
> >
> > switch (type) {
> >
> > - case 0: bits = dev->evbit; len = EV_MAX; break;
> > - case EV_KEY: bits = dev->keybit; len = KEY_MAX; break;
> > - case EV_REL: bits = dev->relbit; len = REL_MAX; break;
> > - case EV_ABS: bits = dev->absbit; len = ABS_MAX; break;
> > - case EV_MSC: bits = dev->mscbit; len = MSC_MAX; break;
> > - case EV_LED: bits = dev->ledbit; len = LED_MAX; break;
> > - case EV_SND: bits = dev->sndbit; len = SND_MAX; break;
> > - case EV_FF: bits = dev->ffbit; len = FF_MAX; break;
> > - case EV_SW: bits = dev->swbit; len = SW_MAX; break;
> > - default: return -EINVAL;
>
> I have the feeling this formatting was deliberate, to make comparing the
> lines easier, see the alignment after colons and semicolons.
Yes, I'd rather kept the code here and in mousedev as is.
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-01-17 22:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-17 11:53 [PATCH 8/9] Fixed indentation violation in switch statement : Aligned the "switch" and its subordinate "case" labels in the same column instead of "double-intending" the "case" labels Shailendra Verma
2015-01-17 15:13 ` Antonio Ospite
2015-01-17 22:15 ` Dmitry Torokhov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).