On Tue, Dec 29, 2009 at 8:29 PM, Iustin Pop <[email protected]> wrote:
> A piece of old code in bdev.py uses a for loop over a single variable
> because we can 'break' out of the loop or exit on the 'else' path. This
> is not a nice usage of the for loop, it should be converted to a
> standard if...elif...else structure.
>
> In the meantime we silence a warning from pylint (it is actually
> invalid, IMHO) and add a TODO.
> ---
>  lib/bdev.py |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)

LGTM
-- 
olive

Reply via email to