Comparison chaining bug

[gvanrossum]

I have this one-liner:

assert 0.0 < 0.5 < 1.0

This gives the following error:

/tmp/x.py, line 1: Unsupported operand types for > ("float" and "bool")

Eh?!?!

[spkersten]

I'm implementing this feature. Parsing of these kind of comparison expressions is done. I'm now working on the type checking part. See my parsecomp branch for progress.

[JukkaL]

I had a quick look at your branch and it's looking good!

Note that the transform-related code is obsolete and doesn't need to be maintained any longer. It could still be useful for other projects, but I'd rather not spend time updating it.

[spkersten]

Thanks. The transform related code is the only part I still have some doubts about (even tough it passes the existing test). Do you want me to leave it out?

I want to add a few unit tests for chain comparisons. So I should be able to create a pull request after a few evenings.

On 8 sep. 2014, at 02:50, Jukka Lehtosalo notifications@github.com wrote:

I had a quick look at your branch and it's looking good!

Note that the transform-related code is obsolete and doesn't need to be maintained any longer. It could still be useful for other projects, but I'd rather not spend time updating it.

—
Reply to this email directly or view it on GitHub.

[JukkaL]

You can leave it out. If some tests fail, you can add a -skip prefix to not run them. Here is an example from test/data/dyncheck-trans-generics.test:

[case testCastWithDynamicAndTypedArguments-skip]

I'm going to remove all transform-related code one of these days.