Re: [PATCH 1/4] platform/x86: fujitsu-laptop: decrease indentation in acpi_fujitsu_hotkey_notify()

2017-01-13 Thread Jonathan Woithe
On Wed, Jan 11, 2017 at 09:59:30AM +0100, Micha?? K??pie?? wrote: > acpi_fujitsu_hotkey_notify() is pretty deeply nested, which hurts > readability. Strip off one level of indentation by returning early when > the event code supplied as argument is not ACPI_FUJITSU_NOTIFY_CODE1. > >

Re: [PATCH 1/4] platform/x86: fujitsu-laptop: decrease indentation in acpi_fujitsu_hotkey_notify()

2017-01-13 Thread Jonathan Woithe
On Wed, Jan 11, 2017 at 09:59:30AM +0100, Micha?? K??pie?? wrote: > acpi_fujitsu_hotkey_notify() is pretty deeply nested, which hurts > readability. Strip off one level of indentation by returning early when > the event code supplied as argument is not ACPI_FUJITSU_NOTIFY_CODE1. > >

[PATCH 1/4] platform/x86: fujitsu-laptop: decrease indentation in acpi_fujitsu_hotkey_notify()

2017-01-11 Thread Michał Kępień
acpi_fujitsu_hotkey_notify() is pretty deeply nested, which hurts readability. Strip off one level of indentation by returning early when the event code supplied as argument is not ACPI_FUJITSU_NOTIFY_CODE1. Signed-off-by: Michał Kępień --- Changes introduced by this patch

[PATCH 1/4] platform/x86: fujitsu-laptop: decrease indentation in acpi_fujitsu_hotkey_notify()

2017-01-11 Thread Michał Kępień
acpi_fujitsu_hotkey_notify() is pretty deeply nested, which hurts readability. Strip off one level of indentation by returning early when the event code supplied as argument is not ACPI_FUJITSU_NOTIFY_CODE1. Signed-off-by: Michał Kępień --- Changes introduced by this patch are best viewed when