Jeff Layton <jlayton@xxxxxxxxxx> wrote: > ...to reduce the extreme indentation. This should introduce no > behavioral changes. > > Cc: stable@xxxxxxxxxx > Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx> In fact, you could go further. In the code you have: > + } else { > + if (!isLargeBuf) { > ... > + } else { > ... > } > } That could be merged and some more indentation removed: } else if (!isLargeBuf) { ... } else { ... } Though I acknowledge you may want to keep it for the logical structure. Anyway: Acked-by: David Howells <dhowells@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html