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

Fixed Issue #171 - added two extra template overloads of operator[] for T* arguments #189

Merged
merged 3 commits into from
Jan 24, 2016

Conversation

twelsby
Copy link
Contributor

@twelsby twelsby commented Jan 23, 2016

Fix #171 by adding two extra template overloads for operator[] for T* arguments. Seems to work well with no apparent side effects. Note that this is dependent on the small change in pull #188 in order for it to pass AppVeyor.

@twelsby twelsby changed the title Issue171 Fixed Issue171 - added two extra template overloads of operator[] for T* arguments Jan 23, 2016
@twelsby twelsby changed the title Fixed Issue171 - added two extra template overloads of operator[] for T* arguments Fixed Issue #171 - added two extra template overloads of operator[] for T* arguments Jan 23, 2016
@nlohmann nlohmann added this to the Release 1.0.1 milestone Jan 24, 2016
nlohmann added a commit that referenced this pull request Jan 24, 2016
Fixed Issue #171 - added two extra template overloads of operator[] for T* arguments
@nlohmann nlohmann merged commit 21a44da into nlohmann:master Jan 24, 2016
@nlohmann
Copy link
Owner

Thanks a lot!

nlohmann added a commit that referenced this pull request Jan 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants