some flake8 cleanup in elliptic curve

[chapoton]

mainly about E701, E702 : one command per line

[chapoton]

New commits:

​c89962a

some cleaning using flake8 in elliptic curves folder

[git]

Branch pushed to git repo; I updated commit sha1. New commits:

​98fc506

fix syntax error

[chapoton]

green bot, please review

[cremona]

It is hardly necessary for a human to review whitespace/stylistic changes if we delegate these to an automatic process. I did see a few changes which are not just w/s. And I did not know that one could write something like "if K is not QQ" instead of "if not K is QQ"!

[chapoton]

Thanks John.

This was done by hand, so the review is needed, but very boring indeed.

Indeed most of the changes are trivial. The idea is to later check this directly before the new code enter sage. But we have to cleanup all the existing code also.

Using x is not y and x not in Z seems to be the recommended way now.

[cremona]

Replying to chapoton:

Thanks John.

This was done by hand, so the review is needed, but very boring indeed.

Not as boring -- and a lot quicker! -- than making the edits. I appreciate that (especially as a good fraction of the changes were in lines I wrote).

Indeed most of the changes are trivial. The idea is to later check this directly before the new code enter sage. But we have to cleanup all the existing code also.

Using x is not y and x not in Z seems to be the recommended way now.