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.

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

closes #3: info_helper::indice_offset was being used out of range #4

Merged
merged 11 commits into from
May 21, 2015
Merged

closes #3: info_helper::indice_offset was being used out of range #4

merged 11 commits into from
May 21, 2015

Conversation

jgsogo
Copy link
Contributor

@jgsogo jgsogo commented May 19, 2015

info_helper::indice_offset was being used out of range here

@jgsogo
Copy link
Contributor Author

jgsogo commented May 19, 2015

First commit closes #3

I've continued cleaning a little bit some pieces of code where enum pos_t can be used instead of int type (will be very useful if it is replaced by enum class) <-- these should be in another branch for a different PR, but... :/

jardon-u added a commit that referenced this pull request May 21, 2015
@jardon-u
Merge pull request #4 from jgsogo/master
46214ba 
closes #3: info_helper::indice_offset was being used out of range
@jardon-u jardon-u merged commit 46214ba into jardon-u:master May 21, 2015
@jardon-u
Copy link
Owner

Looks good. Thanks for contributing !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jgsogo @jardon-u