fixes #8626 by reverting 0d8b81cd
Merge pull request #8627 from rghe/devel

fixes #8626 by reverting 0d8b81cd
Make code more PEP8 compliant (if(x))

Remove useless parentheses, `if (x):` should be `if x:`.
Make code more PEP8 compliant (whitespace)

> Always surround these binary operators with a single space on either
> side: assignment (`=`), augmented assignment (`+=`, `-=`
> etc.), comparisons (`==`, `<`, `>`, ``!=``, `<>`, `<=`,
> `>=`, `in`, `not in`, `is`, `is not`), Booleans (`and`,
> `or`, `not`).
Make docker inventory plugin more Pythonic

There's no need to check if kwargs is empty before iterating over it.
Refactor CronTab.is_empty
Make code more PEP8 compliant (if len(seq))

> For sequences, (strings, lists, tuples), use the fact that empty
> sequences are false.
>
>     Yes: if not seq:
>          if seq:
>
>     No: if len(seq)
>         if not len(seq)
Subversion._exec: rename args parameter to args_list

`args` is usually used for optional parameters as `*args`. This is not
the case here.

Also the hg and bzr modules use `args_list` instead of `args`, so this
will bring consistency.