Menu

#40 JSON_FAIL_MESSAGE no defined in amalgamated cpp file

0.5.0
closed-accepted
nobody
5
2015-03-06
2011-11-09
Stephen Chu
No

The amalgamated files created from the current trunk use JSON_FAIL_MESSAGE from json/assertions.h. This causes compilation errors.

Discussion

  • Anonymous

    Anonymous - 2012-01-25

    I have attached a patch to this,
    and also adjusted the way jsoncpp.cpp includes its header file...
    the python script would make it include the file like so:
    #include <json/json.h>
    but that does not compile in my environment as the json include folder is not in the global includes, nor does it need to be.

    So I have changed it so the generated .cpp file will say instead:
    #include "json/json.hpp"
    which works perfectly as the compiler will search for the header file relative to where the .cpp file was placed.

    For the assertions.h header change,
    It is different from Francis Bolduc's patch, which can be found here:
    http://sourceforge.net/mailarchive/message.php?msg_id=28671263

    Francis puts assertions.h before config.h, but assertions.h normally wants config.h to be included so it should come second.

     
  • Anonymous

    Anonymous - 2012-01-25

    Now I can't attach files. I'll open a new bug.

     
  • Christopher Dunn

    We have accepted that change at: https://github.com/open-source-parsers/jsoncpp/

     
  • Christopher Dunn

    • status: open --> closed-accepted
    • Group: --> 0.5.0
     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.