print to py3 in data_structures, repl, quadratic_form and dev folders

[fchapoton]

another small step towards python3 print, getting close to the last one

CC: @jdemeyer @embray @jm58660 @tscrim

Component: python3

Author: Frédéric Chapoton

Branch/Commit: b77f47c

Reviewer: Jori Mäntysalo

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

[fchapoton]

Commit: 931d162

[fchapoton]

Branch: public/20643

[fchapoton]

New commits:

931d162

python3 print in data_structures, dev, quadratic_forms and repl

[sagetrac-git]

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

b49501e

python3 print, details

[sagetrac-git]

Changed commit from 931d162 to b49501e

[fchapoton]

comment:3

ok, bot is green, except for the 4 darn failings doctests on my bot (having nothing to do with the present ticket), see #20529

please review

[fchapoton]

comment:4

Should I undo the changes to the dev files ? it seems that dev is going to be removed.

[jm58660]

comment:5

Replying to @fchapoton:

Should I undo the changes to the dev files ? it seems that dev is going to be removed.

In what ticket?

[fchapoton]

comment:6

#20645

[jm58660]

comment:7

Now #20645 has already been closed, so it seems clear that you should "rebase" this ticket, i.e. remove dev files.

[sagetrac-git]

Changed commit from b49501e to d27d324

[sagetrac-git]

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

57cf842	Merge branch 'public/20643' into 7.3.b0
d27d324	undo any changes to src/sage/dev

[fchapoton]

comment:9

done. There may still be a conflict in the next beta, we will see.

[jm58660]

comment:10

OK, I'll review.

[jm58660]

Reviewer: Jori Mäntysalo

[jm58660]

comment:11

These are not equivalent due to a space:

print "L(" + str(1+2*i) + ", (-4/.)): ", RR . . .
print("L({}, (-4/.)): {}".format(1+2*i, RR . . .

Otherwise I see no problems and I am now compiling and testing. However, a question: what is the meaning of the comma at print('key %s' % (k,))?

[jm58660]

comment:12

Tests passed and documentation built. Please read my message "Testing and consecutive spaces in output" on sage-devel before changing this to positive_review on behalf of me. Otherwise I found nothing to complain about.

[sagetrac-git]

Changed commit from d27d324 to b77f47c

[sagetrac-git]

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

b77f47c

trac 20643 python3 print in quadratic forms, detail

[jm58660]

comment:14

Ready to go.

[vbraun]

Changed branch from public/20643 to b77f47c