On Mon, Mar 28, 2022 at 02:28:56PM +0300, Andy Shevchenko wrote: > In the snippets like the following > > if (...) > return / goto / break / continue ...; > else > ... > > the 'else' is redundant. Get rid of it by using switch-case pattern. > Note that the pattern seems more usual for IOCTL handlers. It appears that part 1 (as per subject) was never submitted before. So, I unify them in v2 of this patch. -- With Best Regards, Andy Shevchenko
Re: [PATCH v1 1/1] gpiolib: Get rid of redundant 'else' (part 2)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH v1 1/1] gpiolib: Get rid of redundant 'else' (part 2)
- From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
- Date: Wed, 30 Mar 2022 18:00:44 +0300
- In-reply-to: <20220328112856.404-1-andriy.shevchenko@linux.intel.com>
- Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
- References: <20220328112856.404-1-andriy.shevchenko@linux.intel.com>
- References:
- [PATCH v1 1/1] gpiolib: Get rid of redundant 'else' (part 2)
- From: Andy Shevchenko
- [PATCH v1 1/1] gpiolib: Get rid of redundant 'else' (part 2)
- Prev by Date: [PATCH v1 1/3] gpiolib: Split out for_each_gpio_desc() macro
- Next by Date: [PATCH v2 1/1] gpiolib: Get rid of redundant 'else'
- Previous by thread: [PATCH v1 1/1] gpiolib: Get rid of redundant 'else' (part 2)
- Next by thread: [GIT PULL] pin control bulk changes for v5.18
- Index(es):
 |