Re: [PATCH] staging: comedi: db_pcidas64: invert conditional

2017-02-27 Thread Tobin C. Harding
On Mon, Feb 27, 2017 at 10:46:34AM +, Ian Abbott wrote: > On 27/02/17 06:36, Tobin C. Harding wrote: > >Checkpatch emits WARNING: Avoid multiple line dereference. It is > >possible to reduce the level of indentation by inverting a conditional > >and continuing loop if new conditional evaluates

Re: [PATCH] staging: comedi: db_pcidas64: invert conditional

2017-02-27 Thread Ian Abbott
On 27/02/17 06:36, Tobin C. Harding wrote: Checkpatch emits WARNING: Avoid multiple line dereference. It is possible to reduce the level of indentation by inverting a conditional and continuing loop if new conditional evaluates to true. Invert conditional. Continue loop if new conditional evalua

[PATCH] staging: comedi: db_pcidas64: invert conditional

2017-02-26 Thread Tobin C. Harding
Checkpatch emits WARNING: Avoid multiple line dereference. It is possible to reduce the level of indentation by inverting a conditional and continuing loop if new conditional evaluates to true. Invert conditional. Continue loop if new conditional evaluates to true. Reduce subsequent code indentati