Skip to content
This repository was archived by the owner on Jun 3, 2018. It is now read-only.

Add Python 3 support.#1

Open
progval wants to merge 1 commit intolukasmartinelli:masterfrom
progval:python-3
Open

Add Python 3 support.#1
progval wants to merge 1 commit intolukasmartinelli:masterfrom
progval:python-3

Conversation

@progval
Copy link

@progval progval commented Jan 18, 2016

No description provided.

@lukasmartinelli
Copy link
Owner

Wow very nice. Will test it.
Tried supporting Python 3 before but didn't get Python 3 and Python 2 to work simultaneously.

@progval
Copy link
Author

progval commented Jan 18, 2016

Hmm, I just noticed (thanks to Wercker tests) that the list() I added breaks some tests.
Not sure what to do with that

@lukasmartinelli
Copy link
Owner

Hmm, I just noticed (thanks to Wercker tests) that the list() I added breaks some tests.
Not sure what to do with that

It tries to compile the Python tests to C++ as well and I don't support list yet.
I would have to implement a list equivalent in C++ that unwraps the iterator.

The whole range thing I implemented is quite a hack. I am using this library https://github.com/aduchene/rangepp/blob/master/range.hpp

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants