Messages in this thread | | | Date | Thu, 26 Sep 2013 09:55:18 +0200 | From | Daniel Lezcano <> | Subject | Re: [PATCH 11/21] cpuidle: replace multiline statements with single line in cpuidle_idle_call() |
| |
On 09/26/2013 07:51 AM, Viresh Kumar wrote: > On 26 September 2013 03:33, Daniel Lezcano <daniel.lezcano@linaro.org> wrote: >> I splitted these lines because they have 81 cols. > >>> - clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_ENTER, >>> - &dev->cpu); >>> + clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_ENTER, &dev->cpu); > > This one comes in 80 columns > >>> - clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_EXIT, >>> - &dev->cpu); >>> + clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_EXIT, &dev->cpu); > > And this one in 79.. > > And so I did this change.. I have checked it again now..
Well I have 81 with Vi and 80 with Emacs :)
May be some tabs converted to spaces ...
Anyway if it fits 80 cols then it is ok.
-- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
-- 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/
| |