Thanks to visit codestin.com
Credit goes to github.com

Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: SwintonStreet/jsoncpp
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: open-source-parsers/jsoncpp
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 6 commits
  • 12 files changed
  • 8 contributors

Commits on Nov 11, 2025

  1. Fix "include what you use" issue (open-source-parsers#1625)

    Fix IWYU for istreambuf_iterator.
    
    Co-authored-by: Victor Hugo Vianna Silva <[email protected]>
    victorvianna and Victor Hugo Vianna Silva authored Nov 11, 2025
    Configuration menu
    Copy the full SHA
    9af09c4 View commit details
    Browse the repository at this point in the history

Commits on Nov 12, 2025

  1. Make the build configuration under Bazel more correct. (open-source-p…

    …arsers#1600)
    
    * Expose JSON_USE_EXCEPTION and JSON_HAS_INT64 as Bazel config flags with defaults that match the existing Bazel build.
    Switch //:jsoncpp from using copts ro defines for JSON_USE_EXCEPTION and JSON_HAS_INT64 so that rules that depend on it get the same config.
    Make src/test_lib_json/fuzz.cpp respect JSON_USE_EXCEPTION.
    
    * #ifdef stuff that should only be used with JSON_USE_EXCEPTION.
    
    ---------
    
    Co-authored-by: Jordan Bayles <[email protected]>
    bcsgh and baylesj authored Nov 12, 2025
    Configuration menu
    Copy the full SHA
    6f6aca1 View commit details
    Browse the repository at this point in the history
  2. Add Bazel tests (open-source-parsers#1601)

    * Expose JSON_USE_EXCEPTION and JSON_HAS_INT64 as Bazel config flags with defaults that match the existing Bazel build.
    Switch //:jsoncpp from using copts ro defines for JSON_USE_EXCEPTION and JSON_HAS_INT64 so that rules that depend on it get the same config.
    Make src/test_lib_json/fuzz.cpp respect JSON_USE_EXCEPTION.
    
    * #ifdef stuff that should only be used with JSON_USE_EXCEPTION.
    
    * Modify runjsontests.py to allow passing a single test case file.
    
    * Add tests to the Bazel builds.
    
    * Reverse the polarity to fix a bug.
    
    ---------
    
    Co-authored-by: Jordan Bayles <[email protected]>
    bcsgh and baylesj authored Nov 12, 2025
    Configuration menu
    Copy the full SHA
    4bcbc6a View commit details
    Browse the repository at this point in the history
  3. Return false in Reader::readValue when stack limit is exceeded (open-…

    …source-parsers#1619)
    
    jsoncpp, as a shared library, should not call `abort` merely because there's an error reading a value.
    
    See https://en.cppreference.com/w/c/program/abort, `abort` should only be called to **abnormally** cause the program to exit. Functions inserted by `atexit` are also not called, meaning that the host program may have not cleaned up resources properly. But here, exceeding stack limit isn't a sign of abnormalty.
    
    `exit` is not a good substitute either, see the `exit-in-shared-library` from Debian:
    https://lintian.debian.org/tags/exit-in-shared-library.html
    
    Fix open-source-parsers#1618
    
    In this case, returning false seems like a better idea.
    
    Co-authored-by: Jordan Bayles <[email protected]>
    xuhdev and baylesj authored Nov 12, 2025
    Configuration menu
    Copy the full SHA
    30e7528 View commit details
    Browse the repository at this point in the history
  4. Remove deprecated/removed clang-tidy key AnalyzeTemporaryDtors (open-…

    …source-parsers#1614) (open-source-parsers#1615)
    
    Co-authored-by: Ben Magistro <[email protected]>
    Co-authored-by: Jordan Bayles <[email protected]>
    3 people authored Nov 12, 2025
    Configuration menu
    Copy the full SHA
    e16663c View commit details
    Browse the repository at this point in the history
  5. [docs] Consuming JSONCpp via Conan package manager (open-source-parse…

    …rs#1622)
    
    * Fix Conan badge URL
    
    Signed-off-by: Uilian Ries <[email protected]>
    
    * Add Conan instructions in README
    
    Signed-off-by: Uilian Ries <[email protected]>
    
    * Add build missing
    
    Signed-off-by: Uilian Ries <[email protected]>
    
    ---------
    
    Signed-off-by: Uilian Ries <[email protected]>
    Co-authored-by: Jordan Bayles <[email protected]>
    uilianries and baylesj authored Nov 12, 2025
    Configuration menu
    Copy the full SHA
    b511d9e View commit details
    Browse the repository at this point in the history
Loading