Fixes parsing of const defs with separators #134

Merged
merged 1 commit into from May 29, 2015

Conversation

Projects
None yet
3 participants
@laserson
Contributor

laserson commented May 29, 2015

The Thrift IDL grammar specifies that constant definitions may be terminated with a separator, which was not included in the thriftpy parser.

Uri Laserson
Fixes parsing of const defs with separators
The Thrift IDL grammar specifies that constant definitions may be terminated with a separator, which was not included in the thriftpy parser.
@hit9

This comment has been minimized.

Show comment
Hide comment
@hit9

hit9 May 29, 2015

Contributor

👍 Thank you!

Contributor

hit9 commented May 29, 2015

👍 Thank you!

hit9 added a commit that referenced this pull request May 29, 2015

Merge pull request #134 from laserson/const_with_sep
Fixes parsing of const defs with separators

@hit9 hit9 merged commit f3db6de into eleme:develop May 29, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laserson

This comment has been minimized.

Show comment
Hide comment
@laserson

laserson May 29, 2015

Contributor

Any chance you could cut a release soon?

Contributor

laserson commented May 29, 2015

Any chance you could cut a release soon?

@lxyu

This comment has been minimized.

Show comment
Hide comment
@lxyu

lxyu May 29, 2015

Contributor

@laserson v0.3.1 released ;)

Contributor

lxyu commented May 29, 2015

@laserson v0.3.1 released ;)

@laserson

This comment has been minimized.

Show comment
Hide comment
@laserson

laserson May 29, 2015

Contributor

Thanks, @lxyu!

Contributor

laserson commented May 29, 2015

Thanks, @lxyu!

@pyup-bot pyup-bot referenced this pull request in scieloorg/opac_proc Aug 18, 2016

Closed

Initial Update #10

@pyup-bot pyup-bot referenced this pull request in scieloorg/scielo-manager Aug 26, 2016

Closed

Pin thriftpy to latest version 0.3.9 #1314

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