Skip to content

Commit

Permalink
[JENKINS-56466] SpotBugs cannot tell that checkId is only called with…
Browse files Browse the repository at this point in the history
… the FileLogStorage monitor already held
  • Loading branch information
dwnusbaum committed Jul 18, 2023
1 parent e929275 commit b763772
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,9 @@ public static synchronized LogStorage forFile(File log) {
private final File index;
@SuppressFBWarnings(value = "IS2_INCONSISTENT_SYNC", justification = "actually it is always accessed within the monitor")
private FileOutputStream os;
@SuppressFBWarnings(value = "IS2_INCONSISTENT_SYNC", justification = "actually it is always accessed within the monitor")
private long osStartPosition;
@SuppressFBWarnings(value = "IS2_INCONSISTENT_SYNC", justification = "actually it is always accessed within the monitor")
private CountingOutputStream cos;
@SuppressFBWarnings(value = "IS2_INCONSISTENT_SYNC", justification = "we only care about synchronizing writes")
private OutputStream bos;
Expand Down

0 comments on commit b763772

Please sign in to comment.