The GitHub Actions job "Java CI with Gradle" on poi.git/trunk has succeeded.
Run started by GitHub user pjfanning (triggered by pjfanning).

Head commit for run:
eb061da3c32a26c29f62825a4895d2ac9b1f046b / PJ Fanning 
<[email protected]>
Fix SpotBugs warnings; DocumentInputStream skip/read allocations (#1370)

* Fix SpotBugs inconsistent-synchronization warnings

- ClassIDPredefined: read classId inside the lock
- CellDateFormatter: parse state lives in the static DatePartHandler and is
  copied into final fields, so formatValue reads only immutable state
- DocumentInputStream: drop synchronized from mark/reset; the stream is not
  thread-safe and read/skip never locked

Co-Authored-By: Claude Opus 5.5 <[email protected]>

* HexDump.dump: drop needless static synchronized (SpotBugs US_OBJECT)

The method only delegates to an unsynchronized overload that builds a local
String, so the class-level lock protected nothing.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

* DocumentInputStream: skip without allocating, decode primitives in place

- skip(n) advances through the blocks instead of reading the skipped bytes
  into a throwaway array as large as the skip
- readShort/readInt/readLong/readByte (and read()) decode straight from the
  current block when the value does not span a block boundary, falling back
  to the temporary array only at a boundary

Co-Authored-By: Claude Opus 5.5 <[email protected]>

---------

Co-authored-by: Claude Opus 5.5 <[email protected]>

Report URL: https://github.com/apache/poi/actions/runs/38133448056

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to