Skip to content

add long type support#21

Merged
quandyfactory merged 1 commit intoquandyfactory:masterfrom
lichenbo:patch-1
Jun 16, 2014
Merged

add long type support#21
quandyfactory merged 1 commit intoquandyfactory:masterfrom
lichenbo:patch-1

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