fix more W391

[fchapoton]

and a few other things:

• not using (object)

• some more precise asserts

Component: refactoring

Author: Frédéric Chapoton

Branch/Commit: 188910c

Reviewer: Martin Rubey

Issue created by migration from https://trac.sagemath.org/ticket/34596

[fchapoton]

Commit: 86648c2

[fchapoton]

New commits:

86648c2

some care for W391 (final empty lines) and a few other things

[fchapoton]

Branch: u/chapoton/34596

[sagetrac-git]

Changed commit from 86648c2 to bb97cf1

[sagetrac-git]

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

bb97cf1

fix detail

[mantepse]

comment:3

Please do not modify stream.py right now, this might hold up the long chain of tickets fixing really evil bugs, culminating in #34552.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

188910c

some care for W391 (final empty lines) and a few other things

[sagetrac-git]

Changed commit from bb97cf1 to 188910c

[fchapoton]

comment:5

ok, now not touching this file at all

[mantepse]

comment:6

In dev_tools.py, it is unclear why you do not replace object in the following line

    for obj in itertools.chain.from_iterable(expand_comma_separated_names(object)
                                             for object in objects):

(although I admit it is unclear to me, why you replace object at all)

[fchapoton]

comment:7

once done, we will be able to check that nobody uses the obsolete syntax class C(object): anymore.

[mantepse]

Reviewer: Martin Rubey

[mantepse]

comment:8

Thank you!

[vbraun]

Changed branch from u/chapoton/34596 to 188910c