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

Compiler Errors With JSON.hpp #265

Closed
06needhamt opened this issue Jun 13, 2016 · 5 comments
Closed

Compiler Errors With JSON.hpp #265

06needhamt opened this issue Jun 13, 2016 · 5 comments
Assignees
Labels
Milestone

Comments

@06needhamt
Copy link
Contributor

06needhamt commented Jun 13, 2016

Hi i am experiencing some compiler errors with JSON.hpp the errors are as follows

Error   C2589   '(': illegal token on right side of '::'    json.hpp 8475   
Error   C2059   syntax error: '::'   json.hpp   8475    
Error   C2143   syntax error: missing '(' before '::'   json.hpp    8475    
Error   C2589   '(': illegal token on right side of '::'    json.hpp    8481    
Error   C2059   syntax error: '::'  json.hpp    8481    
Error   C2143   syntax error: missing '(' before '::'   json.hpp    8481    

This was compiled using Visual Studio 2015 Update 2 on Windows 10 build 10586

Hope this helps fix the issue

@gregmarr
Copy link
Contributor

I'm guessing that this is the same issue as:
#260

If not, please give the text of the line that shows the error, since 8481 in the latest code is a blank line.

@nlohmann nlohmann added the platform: visual studio related to MSVC label Jun 14, 2016
@nlohmann nlohmann self-assigned this Jun 14, 2016
@nlohmann
Copy link
Owner

I agree with @gregmarr.

@06needhamt, could you please try the latest version of the develop branch? I just merged a fix for the related issue #260 which should fix your problem as well.

@nlohmann nlohmann added the state: please discuss please discuss the issue or vote for your favorite option label Jun 14, 2016
@06needhamt
Copy link
Contributor Author

@nlohmann The issue still occurrs for me when using the develop branch with the error now occuring on line 8507.
However i have made a fix and will send PR shortly

@06needhamt
Copy link
Contributor Author

See PR #266

nlohmann added a commit that referenced this issue Jun 14, 2016
@nlohmann
Copy link
Owner

Merged #266. Thanks a lot!

@nlohmann nlohmann removed the state: please discuss please discuss the issue or vote for your favorite option label Jun 14, 2016
@nlohmann nlohmann added this to the Release 2.0.0 milestone Jun 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants