OK that looked good to me. I pushed a small change and I'll merge this as soon as the Travis flake8 build passes.

LGTM, although I've not checked what these particular error codes are.

Thanks @lesteve and @jnothman !

Merged thanks a lot for the PR and the build reproducing the problem!

Thanks! For future reference, the default flake8 errors are (cf [1], [2]),

• E121 (*^) continuation line under-indented for hanging indent
• E123 (*) closing bracket does not match indentation of opening bracket’s line
• E126 (*^) continuation line over-indented for hanging indent
• E226 (*) missing whitespace around arithmetic operator
• E26 : couldn't find where it is documented
• E704 (*) multiple statements on one line (def)
• W503 (*) line break occurred before a binary operator
• W504 : couldn't find where it is documented

the flake8 documentation also states,

In the default configuration, the checks E121, E123, E126, E133, E226, E241, E242, E704 and W503 are ignored because they are not rules unanimously accepted, and PEP 8 does not enforce them.

(which BTW doesn't match with the actual default parameters)