opendev/gerrit
Code Issues Proposed changes

Add "final" to the public static PLC_ORDER.

Browse Source 
Without final this could be changed by external classes.
This was reported by a Fortify code scan at SAP.

Change-Id: Iab765489d83f86187972831fdc34298c2a777a31
This commit is contained in:
Saša Živkov
2015-09-28 16:13:19 +02:00
committed by David Pursehouse
parent 1f5af647d8
commit a709dc1271
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeNotes.java
View File

@@ -76,7 +76,7 @@ public class ChangeNotes extends AbstractChangeNotes<ChangeNotes> {
}
});
public static Comparator<PatchLineComment> PLC_ORDER =
public static final Comparator<PatchLineComment> PLC_ORDER =
new Comparator<PatchLineComment>() {
@Override
public int compare(PatchLineComment c1, PatchLineComment c2) {