py3 print in misc folder

[fchapoton]

another step towards python3 print style, now in the rather large misc folder

CC: @jdemeyer @embray

Component: python3

Author: Frédéric Chapoton

Branch/Commit: 9e3e403

Reviewer: Jori Mäntysalo

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

[fchapoton]

New commits:

89994a3

python3 print in misc folder

[fchapoton]

Branch: public/20603

[fchapoton]

Commit: 89994a3

[jm58660]

Reviewer: Jori Mäntysalo

[jm58660]

comment:2

Why I.sort(); print I and not I.sort(); I? Same with for example print N[len(dir):].

You have usually changed continuation line markers, but not in print(kwds) nor in print(af.fix_to_pos()).

Shouldn't print(" ".join(str(i) for i in f)) be print("".join(str(i) for i in f))? Same in src/sage/misc/misc.py.

I will now compile and test.

[jm58660]

comment:3

My R is broken, so I got errors from misc/sageinspect.py and misc/nested_class.pyx. Otherwise I could have marked this as positive review. I hope that someone else can compile and test those two files.

[sagetrac-git]

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

ca115d3

trac 20603 python print in misc folder, details

[sagetrac-git]

Changed commit from 89994a3 to ca115d3

[fchapoton]

comment:5

These were real failures, due to a line shift in the code.

Let us wait for the next patchbot report

[sagetrac-git]

Changed commit from ca115d3 to 9e3e403

[sagetrac-git]

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

9e3e403

trac 20603 details

[jm58660]

comment:7

Replying to @fchapoton:

These were real failures, due to a line shift in the code.

The one in misc/nested_class.pyx. But from sageinspect.py I got "RuntimeError: unable to start r". I am quite sure that your two line change works, but anyways would be nice to have someone else to check those. Might be that I won't do that for following three days.

[jm58660]

comment:8

Got it compiled and tested.

[vbraun]

Changed branch from public/20603 to 9e3e403