On Tue, Dec 06, 2016 at 10:34:17PM +0100, Iustin Pop wrote: > On 2016-12-06 17:30:22, Ganeti Development List wrote: > > This avoids a pylint too-many-nested-blocks warning. > > > > NB it's hard to see from the diff because of all the whitespace, but > > this just turns the if result.fail_msg check into a guard that continues > > the next iteration of the loop, and dedends the rest of the loop body > > rather than having it as an else: > > Actually this is almost readable, because of the empty lines that break > the patch into multiple chunks.
You're better at this than me then! These kinds of diffs confuse the hell out of me. :) Cheers, Brian.