[issue21634] Pystone uses floats

2014-06-02 Thread Lennart Regebro
Lennart Regebro added the comment: Awesome, thanks! -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue21634] Pystone uses floats

2014-06-02 Thread STINNER Victor
STINNER Victor added the comment: Thanks the patch, I fixed pystone in Python 3.4 and 3.5. -- resolution: -> fixed status: open -> closed versions: -Python 3.1, Python 3.2, Python 3.3 ___ Python tracker _

[issue21634] Pystone uses floats

2014-06-02 Thread Roundup Robot
Roundup Robot added the comment: New changeset 1318324aa93a by Victor Stinner in branch '3.4': Issue #21634: Fix pystone micro-benchmark: use floor division instead of true http://hg.python.org/cpython/rev/1318324aa93a New changeset 95b7acdc0f24 by Victor Stinner in branch 'default': (Merge 3.4)

[issue21634] Pystone uses floats

2014-06-02 Thread Lennart Regebro
Lennart Regebro added the comment: Oups, yes, that's a typo. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue21634] Pystone uses floats

2014-06-02 Thread STINNER Victor
STINNER Victor added the comment: +Unde Python 3 version 1.1 would use the normal division I guess that it's a typo: "Under Python 3 ..."? -- ___ Python tracker

[issue21634] Pystone uses floats

2014-06-02 Thread Lennart Regebro
Lennart Regebro added the comment: Yes, good point, I added this in a new diff. -- Added file: http://bugs.python.org/file35443/pystone12.diff ___ Python tracker ___

[issue21634] Pystone uses floats

2014-06-02 Thread STINNER Victor
STINNER Victor added the comment: According to the name of variables ("IntLoc2 = IntLoc3 // IntLoc1"), I agree that integers should be used. Since the performances can be different between int and float, you should change the version and explain your change in the changelog (in the top docstri

[issue21634] Pystone uses floats

2014-06-02 Thread Lennart Regebro
New submission from Lennart Regebro: Pystone uses some floats in Python 3, while in Python 2 it's all integers. And since it is, as far as I can tell, based on Dhrystone, it should be all ints. After fixing the division in the loop to be a floor division it runs the same as in Python 2. I've v