Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#40 JSON_FAIL_MESSAGE no defined in amalgamated cpp file

open
nobody
5
2011-11-09
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
    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
    2012-01-25

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