Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add long type support #21

Merged
merged 1 commit into from
Jun 16, 2014
Merged

Conversation

lichenbo
Copy link
Contributor

The script will throw an type error exception when it met a long value. I add the long type into the tuple and everything is ok now except it will missing the ending 'L'. But that's exactly what I want.

yet another bug
quandyfactory added a commit that referenced this pull request Jun 16, 2014
@quandyfactory quandyfactory merged commit bc60436 into quandyfactory:master Jun 16, 2014
@quandyfactory
Copy link
Owner

Thank you. I followed up on your change by including a couple of other places in the code where long had to be added to the list of numberish types and adding a try/except block to handle Python 3, which does not include a long type. Then I abstracted the xml type attribute definition into a separate function. I've pushed the newest version of the code to github if you want to grab it.

quandyfactory added a commit that referenced this pull request Jun 17, 2014
… up README to work as markdown and ReStructuredText
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants