Thanks to visit codestin.com
Credit goes to github.com

Skip to content

use namedtuple to describe the types/sizes #41

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.

Sign up for GitHub

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

Jump to bottom
Closed
mgeplf wants to merge 3 commits into from
Closed

use namedtuple to describe the types/sizes #41

mgeplf wants to merge 3 commits into from

Conversation

@mgeplf
Copy link
Contributor

@mgeplf mgeplf commented Apr 1, 2016

  • makes the code much more readable

@rdumusc rdumusc self-assigned this Apr 1, 2016
@rdumusc
Copy link

rdumusc commented Apr 1, 2016

Thanks for the tip. I will not merge this PR however because I have too many incompatible changes pending, but will try to port it back later.

@mgeplf
Copy link
Contributor Author

mgeplf commented Apr 1, 2016

Yup, no problem. It's pretty simple change, that I had already done.

Cool, I will close it down.

@mgeplf mgeplf closed this Apr 1, 2016
rdumusc pushed a commit to rdumusc/ZeroBuf that referenced this pull request Apr 4, 2016
Minor style cleanups, including HBPVIS#41
3c81e51
rdumusc pushed a commit to rdumusc/ZeroBuf that referenced this pull request Apr 4, 2016
Minor style cleanups, including HBPVIS#41
b09df00
rdumusc pushed a commit to rdumusc/ZeroBuf that referenced this pull request Apr 5, 2016
Minor style cleanups, including HBPVIS#41
f366a8e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@rdumusc rdumusc

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mgeplf @rdumusc