• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

blues / note-c / 7617859606
93%

Build:
DEFAULT BRANCH: master
Ran 22 Jan 2024 10:02PM UTC
Jobs 1
Files 9
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

22 Jan 2024 09:59PM UTC coverage: 90.635% (+0.003%) from 90.632%
7617859606

push

github

web-flow
Improve cJSON number handling. (#132)

- Get rid of CJSON_NO_CLIB and its associated code paths, per Ray's request.
- Change the type of the J object's valueint member. It should be int32_t if
NOTE_LOWMEM and int64_t otherwise. Capture this in a new #define for the type,
JINTEGER.
- When parsing a JSON number from a JSON string, if the number is within the
bounds of the JINTEGER type, convert it to an integer using JAtoI rather than
simply casting valuenumber to JINTEGER, which can lose precision (e.g. for a
Unix timestamp). If the number is outside the bounds of a JINTEGER, saturate,
which is the same behavior as before this commit.
- When printing a JSON number, print it as an integer if possible. Otherwise,
print it as floating point.
- Fix an issue where JItoA would fail to convert the minimum long int value
(i.e. LONG_MIN) to a string.
- Add a slew of new unit tests to make sure number handling behaves as expected.

988 of 1201 branches covered (0.0%)

Branch coverage included in aggregate %.

2109 of 2216 relevant lines covered (95.17%)

81.64 hits per line

Jobs
ID Job ID Ran Files Coverage
1 7617859606.1 22 Jan 2024 10:02PM UTC 0
90.64
GitHub Action Run
Source Files on build 7617859606
Detailed source file information is not available for this build.
  • Back to Repo
  • 86aab4a6 on github
  • Prev Build on master (#7617388538)
  • Next Build on master (#7635387781)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc