[SLOF] [PATCH v2 2/2] usb/storage: Implement block write support

Laurent Vivier lvivier at redhat.com
Thu Dec 13 19:13:55 AEDT 2018


On 12/12/2018 22:10, Segher Boessenkool wrote:
> On Wed, Dec 12, 2018 at 09:14:21PM +0100, Thomas Huth wrote:
>> On 2018-12-12 15:31, Laurent Vivier wrote:
>> There is another "ELSE FALSE EXIT THEN" at the end of this function. I
>> think you've got to drop the "dir" stack item in that case, too, don't you?
> 
> Early exits should be early, not late.  That's the point of early exits:
> it reduces nesting level, and confusion about what is on the stacks when.

In fact, the "EXIT" Thomas is speaking about is at the top of the
function, it's why we have to drop the "dir" stack item: it has not been
used. The latest version of the series is v4.

Thanks,
Laurent



More information about the SLOF mailing list