PMD Results

The following document contains the results of PMD 5.6.1.

Files

org/eclipse/jgit/api/AddCommand.java

Violation	Line
Useless parentheses.	211–212
Unnecessary use of fully qualified name 'FileMode.GITLINK' due to existing static import 'org.eclipse.jgit.lib.FileMode.GITLINK'	212
Useless parentheses.	213–214

org/eclipse/jgit/api/ArchiveCommand.java

Violation	Line
Avoid modifiers which are implied by the context	122–213

org/eclipse/jgit/api/CheckoutCommand.java

Violation	Line
Avoid modifiers which are implied by the context	140–161
Useless parentheses.	576

org/eclipse/jgit/api/CleanCommand.java

Violation	Line
Useless parentheses.	204
Useless parentheses.	205

org/eclipse/jgit/api/CloneCommand.java

Violation	Line
Avoid empty catch blocks	709–712

org/eclipse/jgit/api/CommitCommand.java

Violation	Line
Useless parentheses.	398–399
Useless parentheses.	400
Useless parentheses.	532

org/eclipse/jgit/api/FetchCommand.java

Violation	Line
Useless parentheses.	193–194

org/eclipse/jgit/api/LogCommand.java

Violation	Line
Avoid empty catch blocks	289–292
Avoid empty catch blocks	292–296

org/eclipse/jgit/api/NameRevCommand.java

Violation	Line
Useless parentheses.	284

org/eclipse/jgit/api/RebaseCommand.java

Violation	Line
These nested if statements could be combined	716–721
Useless parentheses.	783–784
Useless parentheses.	783
Useless parentheses.	784
These nested if statements could be combined	1301–1304

org/eclipse/jgit/api/StashApplyCommand.java

Violation	Line
These nested if statements could be combined	375–379

org/eclipse/jgit/api/TagCommand.java

Violation	Line
Avoid unused method parameters such as 'state'.	216

org/eclipse/jgit/api/TransportConfigCallback.java

Violation	Line
Avoid modifiers which are implied by the context	71

org/eclipse/jgit/attributes/Attribute.java

Violation	Line
Avoid modifiers which are implied by the context	70–86

org/eclipse/jgit/attributes/Attributes.java

Violation	Line
Useless parentheses.	159
Useless parentheses.	172
Useless parentheses.	185
Useless parentheses.	199

org/eclipse/jgit/attributes/AttributesHandler.java

Violation	Line
Useless parentheses.	144

org/eclipse/jgit/attributes/AttributesNodeProvider.java

Violation	Line
Avoid modifiers which are implied by the context	66
Avoid modifiers which are implied by the context	79

org/eclipse/jgit/attributes/AttributesProvider.java

Violation	Line
Avoid modifiers which are implied by the context	56

org/eclipse/jgit/attributes/AttributesRule.java

Violation	Line
Avoid empty catch blocks	157–159

org/eclipse/jgit/attributes/FilterCommandFactory.java

Violation	Line
Avoid modifiers which are implied by the context	72–73

org/eclipse/jgit/blame/BlameGenerator.java

Violation	Line
Avoid empty while statements	431–433
Avoid empty if statements	521–525

org/eclipse/jgit/diff/DiffConfig.java

Violation	Line
Avoid modifiers which are implied by the context	62–71

org/eclipse/jgit/diff/DiffEntry.java

Violation	Line
Avoid modifiers which are implied by the context	75–90
Avoid modifiers which are implied by the context	93–99

org/eclipse/jgit/diff/DiffFormatter.java

Violation	Line
Too many static imports may lead to messy code	45–1235
Avoid empty catch blocks	714–716

org/eclipse/jgit/diff/Edit.java

Violation	Line
Avoid modifiers which are implied by the context	67–79

org/eclipse/jgit/diff/MyersDiff.java

Violation	Line
Useless parentheses.	320
Useless parentheses.	442–443
Useless parentheses.	501
Useless parentheses.	549

org/eclipse/jgit/diff/RawTextComparator.java

Violation	Line
Useless parentheses.	83
Useless parentheses.	133
Useless parentheses.	171
Useless parentheses.	206
Useless parentheses.	252

org/eclipse/jgit/diff/SimilarityRenameDetector.java

Violation	Line
Useless parentheses.	366

org/eclipse/jgit/dircache/DirCache.java

Violation	Line
Avoid unused constructor parameters such as 'fs'.	375
Avoid empty catch blocks	440–442

org/eclipse/jgit/dircache/DirCacheBuilder.java

Violation	Line
Avoid using a branching statement as the last in a loop.	181

org/eclipse/jgit/dircache/DirCacheCheckout.java

Violation	Line
Useless parentheses.	233
Useless parentheses.	233
Avoid empty if statements	454–457
Avoid empty if statements	459–464
Useless parentheses.	638
Useless parentheses.	780
Useless parentheses.	781
Useless parentheses.	781
Useless parentheses.	781
Useless parentheses.	877
These nested if statements could be combined	889–898
These nested if statements could be combined	892–897
Useless parentheses.	1120–1121
Avoid empty while statements	1564–1566

org/eclipse/jgit/dircache/DirCacheEntry.java

Violation	Line
Useless parentheses.	122
Useless parentheses.	290
Useless parentheses.	392
Useless parentheses.	392

org/eclipse/jgit/dircache/DirCacheIterator.java

Violation	Line
These nested if statements could be combined	246–252

org/eclipse/jgit/dircache/DirCacheTree.java

Violation	Line
Useless parentheses.	492

org/eclipse/jgit/errors/NoMergeBaseException.java

Violation	Line
Avoid modifiers which are implied by the context	66–85

org/eclipse/jgit/errors/StoredObjectRepresentationNotAvailableException.java

Violation	Line
Avoid unused constructor parameters such as 'otp'.	65
Avoid unused constructor parameters such as 'otp'.	78

org/eclipse/jgit/fnmatch/CharacterHead.java

Violation	Line
Unnecessary final modifier in final class	63–65

org/eclipse/jgit/fnmatch/GroupHead.java

Violation	Line
Unnecessary final modifier in final class	135–142
Unnecessary final modifier in final class	164–166
Unnecessary final modifier in final class	173–175
Unnecessary final modifier in final class	182–184
Unnecessary final modifier in final class	191–193
Unnecessary final modifier in final class	200–202
Unnecessary final modifier in final class	209–211
Unnecessary final modifier in final class	222–224

org/eclipse/jgit/fnmatch/Head.java

Violation	Line
Avoid modifiers which are implied by the context	57

org/eclipse/jgit/fnmatch/RestrictedWildCardHead.java

Violation	Line
Unnecessary final modifier in final class	57–59

org/eclipse/jgit/fnmatch/WildCardHead.java

Violation	Line
Unnecessary final modifier in final class	54–56

org/eclipse/jgit/gitrepo/ManifestParser.java

Violation	Line
Avoid modifiers which are implied by the context	111–112

org/eclipse/jgit/gitrepo/RepoCommand.java

Violation	Line
Avoid modifiers which are implied by the context	153
Avoid modifiers which are implied by the context	169–170
Avoid empty catch blocks	516–518
Useless parentheses.	709
Useless parentheses.	710
Useless parentheses.	737
Useless parentheses.	738

org/eclipse/jgit/hooks/GitHook.java

Violation	Line
Avoid empty catch blocks	162–164

org/eclipse/jgit/ignore/IgnoreNode.java

Violation	Line
Avoid modifiers which are implied by the context	62–79

org/eclipse/jgit/ignore/internal/IMatcher.java

Violation	Line
Avoid modifiers which are implied by the context	53–64

org/eclipse/jgit/ignore/internal/NameMatcher.java

Violation	Line
Ternary operators that can be simplified with \|\| or &&	66
Useless parentheses.	130

org/eclipse/jgit/ignore/internal/PathMatcher.java

Violation	Line
Too many static imports may lead to messy code	43–343
Unnecessary use of fully qualified name 'Strings.getPathSeparator' due to existing static import 'org.eclipse.jgit.ignore.internal.Strings.getPathSeparator'	131

org/eclipse/jgit/ignore/internal/Strings.java

Violation	Line
Useless parentheses.	138
Avoid modifiers which are implied by the context	234–236
These nested if statements could be combined	337–339
Useless parentheses.	398

org/eclipse/jgit/ignore/internal/WildMatcher.java

Violation	Line
Unnecessary final modifier in final class	64–68
Unnecessary final modifier in final class	72–74

org/eclipse/jgit/internal/ketch/ElectionRound.java

Violation	Line
These nested if statements could be combined	106–108

org/eclipse/jgit/internal/ketch/KetchLeader.java

Violation	Line
Too many static imports may lead to messy code	44–642
Avoid modifiers which are implied by the context	129–141

org/eclipse/jgit/internal/ketch/KetchPreReceive.java

Violation	Line
Too many static imports may lead to messy code	44–158

org/eclipse/jgit/internal/ketch/KetchReplica.java

Violation	Line
Too many static imports may lead to messy code	44–791
Invoke equals() on the object you've already ensured is not null	348

org/eclipse/jgit/internal/ketch/KetchSystem.java

Violation	Line
Too many static imports may lead to messy code	44–374

org/eclipse/jgit/internal/ketch/LagCheck.java

Violation	Line
Too many static imports may lead to messy code	44–185
Avoid empty catch blocks	150–152

org/eclipse/jgit/internal/ketch/LocalReplica.java

Violation	Line
Too many static imports may lead to messy code	44–240

org/eclipse/jgit/internal/ketch/Proposal.java

Violation	Line
Too many static imports may lead to messy code	44–448
Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED'	427

org/eclipse/jgit/internal/ketch/RemoteGitReplica.java

Violation	Line
Too many static imports may lead to messy code	44–329

org/eclipse/jgit/internal/ketch/ReplicaConfig.java

Violation	Line
Too many static imports may lead to messy code	44–247

org/eclipse/jgit/internal/revwalk/AddToBitmapFilter.java

Violation	Line
Avoid empty if statements	81–83

org/eclipse/jgit/internal/revwalk/AddUnseenToBitmapFilter.java

Violation	Line
Avoid empty if statements	89–91

org/eclipse/jgit/internal/storage/dfs/DfsBlockCacheConfig.java

Violation	Line
Too many static imports may lead to messy code	44–253

org/eclipse/jgit/internal/storage/dfs/DfsGarbageCollector.java

Violation	Line
Too many static imports may lead to messy code	44–788
Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK'	477

org/eclipse/jgit/internal/storage/dfs/DfsInserter.java

Violation	Line
Avoid empty catch blocks	247–249
Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK'	623
Useless parentheses.	630

org/eclipse/jgit/internal/storage/dfs/DfsObjDatabase.java

Violation	Line
Avoid modifiers which are implied by the context	90–151

org/eclipse/jgit/internal/storage/dfs/DfsPackCompactor.java

Violation	Line
Too many static imports may lead to messy code	44–541

org/eclipse/jgit/internal/storage/dfs/DfsPackDescription.java

Violation	Line
Useless parentheses.	100
Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK'	515

org/eclipse/jgit/internal/storage/dfs/DfsPackFile.java

Violation	Line
Unnecessary final modifier in final class	247–249
Useless parentheses.	564
Useless parentheses.	570
Useless parentheses.	577
Useless parentheses.	583
Useless parentheses.	600
Useless parentheses.	624
Useless parentheses.	709
Useless parentheses.	717
Useless parentheses.	789
Useless parentheses.	837
Useless parentheses.	952
Useless parentheses.	1059

org/eclipse/jgit/internal/storage/dfs/DfsPackParser.java

Violation	Line
Avoid empty catch blocks	183–185

org/eclipse/jgit/internal/storage/dfs/DfsReader.java

Violation	Line
Avoid empty catch blocks	363–365
Avoid empty catch blocks	383–385
Useless parentheses.	785

org/eclipse/jgit/internal/storage/dfs/DfsReaderOptions.java

Violation	Line
Too many static imports may lead to messy code	44–182

org/eclipse/jgit/internal/storage/dfs/DfsRefDatabase.java

Violation	Line
Avoid empty catch blocks	384–386

org/eclipse/jgit/internal/storage/dfs/DfsReftable.java

Violation	Line
Avoid empty catch blocks	176–178

org/eclipse/jgit/internal/storage/dfs/DfsStreamKey.java

Violation	Line
Ensure you override both equals() and hashCode()	141

org/eclipse/jgit/internal/storage/dfs/ReadableChannel.java

Violation	Line
Avoid modifiers which are implied by the context	60
Avoid modifiers which are implied by the context	73
Avoid modifiers which are implied by the context	86
Avoid modifiers which are implied by the context	105
Avoid modifiers which are implied by the context	134

org/eclipse/jgit/internal/storage/dfs/ReftableBatchRefUpdate.java

Violation	Line
Too many static imports may lead to messy code	44–465

org/eclipse/jgit/internal/storage/dfs/ReftableStack.java

Violation	Line
Avoid empty catch blocks	122–124

org/eclipse/jgit/internal/storage/file/BitSet.java

Violation	Line
Unnecessary final modifier in final class	62–64
Unnecessary final modifier in final class	66–74
Unnecessary final modifier in final class	76–80
Unnecessary final modifier in final class	82–85
Unnecessary final modifier in final class	87–112
Unnecessary final modifier in final class	114–116
Unnecessary final modifier in final class	118–120

org/eclipse/jgit/internal/storage/file/BitmapIndexImpl.java

Violation	Line
Unnecessary final modifier in final class	360–362

org/eclipse/jgit/internal/storage/file/CachedObjectDirectory.java

Violation	Line
Avoid empty catch blocks	112–114
These nested if statements could be combined	199–201

org/eclipse/jgit/internal/storage/file/FileObjectDatabase.java

Violation	Line
Avoid modifiers which are implied by the context	63–65

org/eclipse/jgit/internal/storage/file/FileRepository.java

Violation	Line
These nested if statements could be combined	237–240
Avoid empty catch blocks	331–333

org/eclipse/jgit/internal/storage/file/FileSnapshot.java

Violation	Line
Useless parentheses.	198

org/eclipse/jgit/internal/storage/file/GC.java

Violation	Line
Avoid empty catch blocks	420–422
Avoid empty catch blocks	457–459
Avoid empty catch blocks	573–576
Avoid empty catch blocks	953–955
Useless parentheses.	973–975
Avoid empty catch blocks	978–980
Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX'	1148
Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX'	1151

org/eclipse/jgit/internal/storage/file/GcLog.java

Violation	Line
Avoid empty catch blocks	111–113

org/eclipse/jgit/internal/storage/file/InflatingBitSet.java

Violation	Line
Unnecessary final modifier in final class	76–80
Unnecessary final modifier in final class	82–124
Unnecessary final modifier in final class	126–129
Unnecessary final modifier in final class	131–133
Unnecessary final modifier in final class	135–137
Unnecessary final modifier in final class	139–141
Unnecessary final modifier in final class	143–147
Unnecessary final modifier in final class	149–153
Unnecessary final modifier in final class	155–162
Unnecessary final modifier in final class	164–166

org/eclipse/jgit/internal/storage/file/LazyObjectIdSetFile.java

Violation	Line
Avoid empty catch blocks	98–100

org/eclipse/jgit/internal/storage/file/LockFile.java

Violation	Line
Avoid empty catch blocks	101–103
Avoid unused constructor parameters such as 'fs'.	155

org/eclipse/jgit/internal/storage/file/ObjectDirectory.java

Violation	Line
These nested if statements could be combined	327–329
These nested if statements could be combined	343–345
Avoid using a branching statement as the last in a loop.	501
Avoid using a branching statement as the last in a loop.	594
Avoid using a branching statement as the last in a loop.	640
Useless parentheses.	655
Useless parentheses.	656
Avoid empty catch blocks	730–732
Useless parentheses.	776

org/eclipse/jgit/internal/storage/file/ObjectDirectoryPackParser.java

Violation	Line
Avoid empty catch blocks	216–218

org/eclipse/jgit/internal/storage/file/PackBitmapIndex.java

Violation	Line
Avoid empty catch blocks	107–109

org/eclipse/jgit/internal/storage/file/PackBitmapIndexV1.java

Violation	Line
Useless parentheses.	163

org/eclipse/jgit/internal/storage/file/PackFile.java

Violation	Line
Useless parentheses.	445
Useless parentheses.	451
Useless parentheses.	458
Useless parentheses.	464
Useless parentheses.	481
Useless parentheses.	514
Useless parentheses.	534
Useless parentheses.	591
Useless parentheses.	599
Avoid empty catch blocks	714–718
Useless parentheses.	849
Useless parentheses.	898
Useless parentheses.	1013
Useless parentheses.	1119
Useless parentheses.	1205

org/eclipse/jgit/internal/storage/file/PackIndex.java

Violation	Line
Avoid empty catch blocks	109–111

org/eclipse/jgit/internal/storage/file/PackIndexV1.java

Violation	Line
Useless parentheses.	240

org/eclipse/jgit/internal/storage/file/PackIndexV2.java

Violation	Line
Useless parentheses.	231
Useless parentheses.	338

org/eclipse/jgit/internal/storage/file/PackInserter.java

Violation	Line
Avoid empty catch blocks	329–331
Avoid empty catch blocks	336–338
Avoid empty catch blocks	341–343

org/eclipse/jgit/internal/storage/file/PackLock.java

Violation	Line
Avoid unused constructor parameters such as 'fs'.	68

org/eclipse/jgit/internal/storage/file/PackedBatchRefUpdate.java

Violation	Line
Too many static imports may lead to messy code	44–551
Avoid using a branching statement as the last in a loop.	356
Useless parentheses.	393
Useless parentheses.	394
Avoid empty catch blocks	451–453
Avoid empty catch blocks	472–486
Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED'	546

org/eclipse/jgit/internal/storage/file/RefDirectory.java

Violation	Line
Too many static imports may lead to messy code	47–1490
Avoid using a branching statement as the last in a loop.	516
These nested if statements could be combined	538–539
Useless parentheses.	1162
Useless parentheses.	1177
Unnecessary use of fully qualified name 'Constants.HEAD' due to existing static import 'org.eclipse.jgit.lib.Constants.HEAD'	1214
Unnecessary use of fully qualified name 'Constants.R_REFS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_REFS'	1245
Avoid modifiers which are implied by the context	1384–1388

org/eclipse/jgit/internal/storage/file/RefDirectoryRename.java

Violation	Line
Avoid empty catch blocks	217–219

org/eclipse/jgit/internal/storage/file/ReflogWriter.java

Violation	Line
Too many static imports may lead to messy code	46–270
Useless parentheses.	226

org/eclipse/jgit/internal/storage/file/UnpackedObjectCache.java

Violation	Line
Avoid empty if statements	69–73

org/eclipse/jgit/internal/storage/file/WindowCursor.java

Violation	Line
Useless parentheses.	319

org/eclipse/jgit/internal/storage/io/BlockSource.java

Violation	Line
Avoid empty catch blocks	136–138

org/eclipse/jgit/internal/storage/pack/DeltaIndexScanner.java

Violation	Line
Useless parentheses.	68

org/eclipse/jgit/internal/storage/pack/DeltaTask.java

Violation	Line
This for loop could be simplified to a while loop	140–150
Unnecessary final modifier in final class	267–269
Ternary operators that can be simplified with \|\| or &&	355

org/eclipse/jgit/internal/storage/pack/DeltaWindow.java

Violation	Line
Avoid using a branching statement as the last in a loop.	225
Avoid empty catch blocks	309–311

org/eclipse/jgit/internal/storage/pack/DeltaWindowEntry.java

Violation	Line
Unnecessary final modifier in final class	57–61
Unnecessary final modifier in final class	64–66
Unnecessary final modifier in final class	69–71
Unnecessary final modifier in final class	74–76
Unnecessary final modifier in final class	79–81
Unnecessary final modifier in final class	83–93

org/eclipse/jgit/internal/storage/pack/ObjectReuseAsIs.java

Violation	Line
Avoid modifiers which are implied by the context	82
Avoid modifiers which are implied by the context	117–119
Avoid modifiers which are implied by the context	158–159
Avoid modifiers which are implied by the context	203–205
Avoid modifiers which are implied by the context	219–220
Avoid modifiers which are implied by the context	237–238

org/eclipse/jgit/internal/storage/pack/PackOutputStream.java

Violation	Line
Unnecessary final modifier in final class	110–114
Unnecessary final modifier in final class	118–138
Unnecessary final modifier in final class	146–153
Unnecessary final modifier in final class	170–172
Unnecessary final modifier in final class	190–205
Useless parentheses.	193
Unnecessary final modifier in final class	207–216
Unnecessary final modifier in final class	218–225
Unnecessary final modifier in final class	227–232
Unnecessary final modifier in final class	239–241
Unnecessary final modifier in final class	252–254
Unnecessary final modifier in final class	257–259

org/eclipse/jgit/internal/storage/pack/PackWriter.java

Violation	Line
Too many static imports may lead to messy code	45–2609
Avoid empty while statements	1849–1851
These nested if statements could be combined	2129–2130
Useless parentheses.	2145
Useless parentheses.	2146
Useless parentheses.	2514
Useless parentheses.	2531
Avoid modifiers which are implied by the context	2557–2575

org/eclipse/jgit/internal/storage/pack/PackWriterBitmapPreparer.java

Violation	Line
Useless parentheses.	137
Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE'	220
Useless parentheses.	253
Useless parentheses.	254–255
Useless parentheses.	255
Useless parentheses.	256
Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE'	267
Avoid empty while statements	275–278
These nested if statements could be combined	286–289

org/eclipse/jgit/internal/storage/reftable/BlockReader.java

Violation	Line
Too many static imports may lead to messy code	44–589
Useless parentheses.	561
Useless parentheses.	573

org/eclipse/jgit/internal/storage/reftable/BlockWriter.java

Violation	Line
Too many static imports may lead to messy code	44–605
Useless parentheses.	138

org/eclipse/jgit/internal/storage/reftable/ReftableOutputStream.java

Violation	Line
Too many static imports may lead to messy code	44–249

org/eclipse/jgit/internal/storage/reftable/ReftableReader.java

Violation	Line
Too many static imports may lead to messy code	44–698
Useless parentheses.	172
Useless parentheses.	447
Avoid using a branching statement as the last in a loop.	507
Avoid using a branching statement as the last in a loop.	567
Invoke equals() on the object you've already ensured is not null	676

org/eclipse/jgit/internal/storage/reftable/ReftableWriter.java

Violation	Line
Too many static imports may lead to messy code	44–835
Avoid using a branching statement as the last in a loop.	683

org/eclipse/jgit/internal/storage/reftree/Command.java

Violation	Line
Too many static imports may lead to messy code	44–342

org/eclipse/jgit/internal/storage/reftree/RefTree.java

Violation	Line
Too many static imports may lead to messy code	44–417

org/eclipse/jgit/internal/storage/reftree/RefTreeBatch.java

Violation	Line
Too many static imports may lead to messy code	44–227

org/eclipse/jgit/internal/storage/reftree/Scanner.java

Violation	Line
Too many static imports may lead to messy code	44–286

org/eclipse/jgit/lib/AbbreviatedObjectId.java

Violation	Line
Unnecessary final modifier in final class	77–87
Unnecessary final modifier in final class	101–109
Unnecessary final modifier in final class	122–125
Unnecessary final modifier in final class	134–139
Unnecessary final modifier in final class	141–153
Unnecessary final modifier in final class	155–166
Unnecessary final modifier in final class	247–267
Unnecessary final modifier in final class	283–303
Unnecessary final modifier in final class	319–339
Unnecessary final modifier in final class	346–348
Unnecessary final modifier in final class	376–397

org/eclipse/jgit/lib/AnyObjectId.java

Violation	Line
Ternary operators that can be simplified with \|\| or &&	278

org/eclipse/jgit/lib/AsyncObjectLoaderQueue.java

Violation	Line
Avoid modifiers which are implied by the context	81
Avoid modifiers which are implied by the context	90
Avoid modifiers which are implied by the context	97
Avoid modifiers which are implied by the context	118

org/eclipse/jgit/lib/AsyncObjectSizeQueue.java

Violation	Line
Avoid modifiers which are implied by the context	76
Avoid modifiers which are implied by the context	85
Avoid modifiers which are implied by the context	92
Avoid modifiers which are implied by the context	99

org/eclipse/jgit/lib/AsyncOperation.java

Violation	Line
Avoid modifiers which are implied by the context	71
Avoid modifiers which are implied by the context	76

org/eclipse/jgit/lib/BaseRepositoryBuilder.java

Violation	Line
Too many static imports may lead to messy code	44–783
Useless parentheses.	113
Avoid empty catch blocks	554–556

org/eclipse/jgit/lib/BitmapIndex.java

Violation	Line
Avoid modifiers which are implied by the context	77–117
Avoid modifiers which are implied by the context	123–224

org/eclipse/jgit/lib/BlobObjectChecker.java

Violation	Line
Avoid modifiers which are implied by the context	58–69

org/eclipse/jgit/lib/CheckoutEntry.java

Violation	Line
Avoid modifiers which are implied by the context	15
Avoid modifiers which are implied by the context	22

org/eclipse/jgit/lib/Config.java

Violation	Line
Avoid modifiers which are implied by the context	1438–1447
Avoid modifiers which are implied by the context	1477–1493

org/eclipse/jgit/lib/ConfigSnapshot.java

Violation	Line
Unnecessary use of fully qualified name 'StringUtils.toLowerCase' due to existing static import 'org.eclipse.jgit.util.StringUtils.toLowerCase'	281

org/eclipse/jgit/lib/CoreConfig.java

Violation	Line
Avoid modifiers which are implied by the context	63–72
Avoid modifiers which are implied by the context	81–90
Avoid modifiers which are implied by the context	97–112
Avoid modifiers which are implied by the context	119–131
Avoid modifiers which are implied by the context	148–153
Avoid modifiers which are implied by the context	160–167

org/eclipse/jgit/lib/DefaultTypedConfigGetter.java

Violation	Line
Avoid empty catch blocks	128–130

org/eclipse/jgit/lib/FileMode.java

Violation	Line
Do not start a literal by 0 unless its an octal value	68
Do not start a literal by 0 unless its an octal value	71
Do not start a literal by 0 unless its an octal value	74
Do not start a literal by 0 unless its an octal value	77
Do not start a literal by 0 unless its an octal value	80
Do not start a literal by 0 unless its an octal value	83
Do not start a literal by 0 unless its an octal value	106
Do not start a literal by 0 unless its an octal value	110
Do not start a literal by 0 unless its an octal value	115
Do not start a literal by 0 unless its an octal value	119
Do not start a literal by 0 unless its an octal value	157

org/eclipse/jgit/lib/GitmoduleEntry.java

Violation	Line
No need to import a type that lives in the same package	45

org/eclipse/jgit/lib/IndexDiff.java

Violation	Line
Avoid modifiers which are implied by the context	107–195
Avoid modifiers which are implied by the context	346
These nested if statements could be combined	473–476
These nested if statements could be combined	512–516
Useless parentheses.	514–515
Useless parentheses.	610–611
Useless parentheses.	610–611
Useless parentheses.	610
Useless parentheses.	621

org/eclipse/jgit/lib/ObjectChecker.java

Violation	Line
Too many static imports may lead to messy code	45–1254
Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH'	563
Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH'	595
Useless parentheses.	928
Useless parentheses.	929
Useless parentheses.	1100–1101
Useless parentheses.	1102
Useless parentheses.	1103

org/eclipse/jgit/lib/ProgressMonitor.java

Violation	Line
Avoid modifiers which are implied by the context	52

org/eclipse/jgit/lib/RebaseTodoLine.java

Violation	Line
Avoid modifiers which are implied by the context	61–118
These nested if statements could be combined	191–194

org/eclipse/jgit/lib/Ref.java

Violation	Line
Avoid modifiers which are implied by the context	64–126
Avoid modifiers which are implied by the context	134
Avoid modifiers which are implied by the context	147
Avoid modifiers which are implied by the context	166
Avoid modifiers which are implied by the context	181
Avoid modifiers which are implied by the context	191
Avoid modifiers which are implied by the context	201
Avoid modifiers which are implied by the context	208
Avoid modifiers which are implied by the context	219

org/eclipse/jgit/lib/RefUpdate.java

Violation	Line
Avoid modifiers which are implied by the context	68–171
These nested if statements could be combined	778–780

org/eclipse/jgit/lib/ReflogEntry.java

Violation	Line
Avoid modifiers which are implied by the context	60
Avoid modifiers which are implied by the context	72
Avoid modifiers which are implied by the context	83
Avoid modifiers which are implied by the context	90
Avoid modifiers which are implied by the context	97
Avoid modifiers which are implied by the context	104
Avoid modifiers which are implied by the context	111
Avoid modifiers which are implied by the context	120

org/eclipse/jgit/lib/ReflogReader.java

Violation	Line
Avoid modifiers which are implied by the context	62
Avoid modifiers which are implied by the context	70
Avoid modifiers which are implied by the context	80
Avoid modifiers which are implied by the context	90–91

org/eclipse/jgit/lib/Repository.java

Violation	Line
Useless parentheses.	824
Useless parentheses.	825
Useless parentheses.	826
These nested if statements could be combined	882–883
Avoid empty catch blocks	1331–1335
Avoid empty catch blocks	1348–1350
Avoid empty catch blocks	1361–1363

org/eclipse/jgit/lib/RepositoryBuilder.java

Violation	Line
Avoid unused imports such as 'java.io.File'	46

org/eclipse/jgit/lib/RepositoryCache.java

Violation	Line
Useless parentheses.	302
Avoid modifiers which are implied by the context	347–368

org/eclipse/jgit/lib/StoredConfig.java

Violation	Line
Overriding method merely calls super	95–97

org/eclipse/jgit/lib/TreeFormatter.java

Violation	Line
Too many static imports may lead to messy code	44–403

org/eclipse/jgit/merge/EolAwareOutputStream.java

Violation	Line
Useless parentheses.	86
Useless parentheses.	94

org/eclipse/jgit/merge/MergeConfig.java

Violation	Line
Avoid empty catch blocks	72–74

org/eclipse/jgit/merge/MergeFormatterPass.java

Violation	Line
Useless parentheses.	75

org/eclipse/jgit/merge/MergeResult.java

Violation	Line
Useless parentheses.	141

org/eclipse/jgit/merge/ResolveMerger.java

Violation	Line
Too many static imports may lead to messy code	47–1312
These nested if statements could be combined	392–394
Useless parentheses.	785–786
Useless parentheses.	785–786
These nested if statements could be combined	785–810
Useless parentheses.	785
These nested if statements could be combined	801–802
Useless parentheses.	848–849

org/eclipse/jgit/merge/ThreeWayMerger.java

Violation	Line
Avoid unused constructor parameters such as 'inCore'.	86

org/eclipse/jgit/nls/GlobalBundleCache.java

Violation	Line
Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException'	50
Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException'	51

org/eclipse/jgit/nls/NLS.java

Violation	Line
Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException'	49
Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException'	50

org/eclipse/jgit/notes/NoteMapMerger.java

Violation	Line
Useless parentheses.	284
Useless parentheses.	286
Invoke equals() on the object you've already ensured is not null	298

org/eclipse/jgit/patch/BinaryHunk.java

Violation	Line
Avoid modifiers which are implied by the context	61–67

org/eclipse/jgit/patch/CombinedFileHeader.java

Violation	Line
Overriding method merely calls super	152–154

org/eclipse/jgit/patch/FileHeader.java

Violation	Line
Too many static imports may lead to messy code	44–651
Avoid modifiers which are implied by the context	107–116
Avoid empty catch blocks	247–250
Unnecessary use of fully qualified name 'RawParseUtils.decode' due to existing static import 'org.eclipse.jgit.util.RawParseUtils.decode'	295

org/eclipse/jgit/patch/FormatError.java

Violation	Line
Avoid modifiers which are implied by the context	56–62

org/eclipse/jgit/patch/Patch.java

Violation	Line
Too many static imports may lead to messy code	44–392

org/eclipse/jgit/revplot/AbstractPlotRenderer.java

Violation	Line
Useless parentheses.	195

org/eclipse/jgit/revplot/PlotWalk.java

Violation	Line
Avoid empty catch blocks	166–168

org/eclipse/jgit/revwalk/AsyncRevObjectQueue.java

Violation	Line
Avoid modifiers which are implied by the context	69

org/eclipse/jgit/revwalk/BitmapWalker.java

Violation	Line
Avoid empty catch blocks	138–141
Avoid empty catch blocks	157–172

org/eclipse/jgit/revwalk/DateRevQueue.java

Violation	Line
Useless parentheses.	121

org/eclipse/jgit/revwalk/DepthWalk.java

Violation	Line
Avoid modifiers which are implied by the context	64
Avoid modifiers which are implied by the context	72
Avoid modifiers which are implied by the context	79
Avoid modifiers which are implied by the context	82–101
Avoid modifiers which are implied by the context	104–181
Avoid modifiers which are implied by the context	184–274

org/eclipse/jgit/revwalk/ObjectWalk.java

Violation	Line
Avoid unused imports such as 'org.eclipse.jgit.lib.AnyObjectId'	60
Do not start a literal by 0 unless its an octal value	86
Do not start a literal by 0 unless its an octal value	87
Do not start a literal by 0 unless its an octal value	88
Do not start a literal by 0 unless its an octal value	89
Avoid using a branching statement as the last in a loop.	324
Avoid empty while statements	740–742

org/eclipse/jgit/revwalk/RevCommitList.java

Violation	Line
Avoid empty catch blocks	193–196
Useless parentheses.	371

org/eclipse/jgit/revwalk/RevWalk.java

Violation	Line
Useless qualified this usage in the same class.	1358

org/eclipse/jgit/revwalk/TopoSortGenerator.java

Violation	Line
Avoid using a branching statement as the last in a loop.	126

org/eclipse/jgit/submodule/SubmoduleWalk.java

Violation	Line
Useless parentheses.	528
Avoid using a branching statement as the last in a loop.	622

org/eclipse/jgit/transport/AdvertiseRefsHook.java

Violation	Line
Avoid modifiers which are implied by the context	58–68
Avoid modifiers which are implied by the context	80–81
Avoid modifiers which are implied by the context	93–94

org/eclipse/jgit/transport/BasePackConnection.java

Violation	Line
Avoid empty catch blocks	331–333
Avoid empty catch blocks	342–344
Avoid empty catch blocks	372–374

org/eclipse/jgit/transport/BasePackFetchConnection.java

Violation	Line
Avoid empty catch blocks	423–425
Avoid empty catch blocks	480–482
Avoid empty catch blocks	500–503
Avoid empty catch blocks	802–804

org/eclipse/jgit/transport/BasePackPushConnection.java

Violation	Line
Avoid empty catch blocks	185–187
Avoid empty catch blocks	191–193
Useless parentheses.	437

org/eclipse/jgit/transport/BaseReceivePack.java

Violation	Line
Too many static imports may lead to messy code	44–1982
Avoid empty catch blocks	380–382
Avoid empty catch blocks	391–393
Avoid empty catch blocks	407–409
Avoid empty catch blocks	1055–1057
Avoid empty catch blocks	1362–1364

org/eclipse/jgit/transport/BundleFetchConnection.java

Violation	Line
Avoid empty catch blocks	266–268
Avoid empty catch blocks	303–305

org/eclipse/jgit/transport/Connection.java

Violation	Line
Avoid modifiers which are implied by the context	71
Avoid modifiers which are implied by the context	85
Avoid modifiers which are implied by the context	98
Avoid modifiers which are implied by the context	118
Avoid modifiers which are implied by the context	135
Avoid modifiers which are implied by the context	144

org/eclipse/jgit/transport/Daemon.java

Violation	Line
Avoid using java.lang.ThreadGroup; it is not thread safe	112
Avoid empty catch blocks	316–318
Avoid empty catch blocks	342–344
Avoid empty catch blocks	426–428
Avoid empty catch blocks	428–430
Avoid empty catch blocks	430–432
Avoid empty catch blocks	435–437
Avoid empty catch blocks	440–442

org/eclipse/jgit/transport/DaemonService.java

Violation	Line
Useless qualified this usage in the same class.	70

org/eclipse/jgit/transport/FetchConnection.java

Violation	Line
Avoid modifiers which are implied by the context	112–114
Avoid modifiers which are implied by the context	154–156
Avoid modifiers which are implied by the context	176
Avoid modifiers which are implied by the context	199
Avoid modifiers which are implied by the context	211
Avoid modifiers which are implied by the context	221

org/eclipse/jgit/transport/FetchProcess.java

Violation	Line
Unnecessary use of fully qualified name 'ReceiveCommand.Result.OK' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.OK'	539

org/eclipse/jgit/transport/FetchResult.java

Violation	Line
Avoid unused imports such as 'java.util.Collection'	50

org/eclipse/jgit/transport/GitProtocolConstants.java

Violation	Line
Avoid modifiers which are implied by the context	218–220

org/eclipse/jgit/transport/HttpAuthMethod.java

Violation	Line
Useless parentheses.	175
Useless parentheses.	176
Avoid empty catch blocks	192–194
Avoid empty if statements	416–418
Avoid empty if statements	419–421

org/eclipse/jgit/transport/HttpConfig.java

Violation	Line
Useless parentheses.	96–114
Avoid empty if statements	392–394

org/eclipse/jgit/transport/InternalFetchConnection.java

Violation	Line
Avoid empty catch blocks	110–112
Avoid empty catch blocks	112–114
Avoid empty catch blocks	123–125
Avoid empty catch blocks	129–131

org/eclipse/jgit/transport/InternalPushConnection.java

Violation	Line
Avoid empty catch blocks	102–104
Avoid empty catch blocks	104–106
Avoid empty catch blocks	106–108
Avoid empty catch blocks	108–110
Avoid empty catch blocks	113–115
Avoid empty catch blocks	119–121

org/eclipse/jgit/transport/JschConfigSessionFactory.java

Violation	Line
Avoid empty catch blocks	367–369
Avoid empty catch blocks	369–371
Avoid empty catch blocks	390–392
These nested if statements could be combined	454–467
Avoid empty catch blocks	464–466

org/eclipse/jgit/transport/JschSession.java

Violation	Line
Avoid empty catch blocks	171–173

org/eclipse/jgit/transport/NetRCCredentialsProvider.java

Violation	Line
These nested if statements could be combined	103–107

org/eclipse/jgit/transport/NonceGenerator.java

Violation	Line
Avoid modifiers which are implied by the context	69–70
Avoid modifiers which are implied by the context	94–95

org/eclipse/jgit/transport/ObjectCountCallback.java

Violation	Line
Avoid unused imports such as 'java.io.OutputStream'	46
Avoid unused imports such as 'org.eclipse.jgit.lib.ProgressMonitor'	48

org/eclipse/jgit/transport/OpenSshConfig.java

Violation	Line
Avoid empty catch blocks	232–234
Avoid empty catch blocks	372–374

org/eclipse/jgit/transport/PackParser.java

Violation	Line
Avoid modifiers which are implied by the context	106–112
Useless parentheses.	851
Useless parentheses.	1026
Avoid empty catch blocks	1184–1188
Avoid empty catch blocks	1204–1208

org/eclipse/jgit/transport/PacketLineIn.java

Violation	Line
Avoid modifiers which are implied by the context	77–88
Avoid empty catch blocks	245–247

org/eclipse/jgit/transport/PostReceiveHook.java

Violation	Line
Avoid modifiers which are implied by the context	66–72
Avoid modifiers which are implied by the context	84–85

org/eclipse/jgit/transport/PostUploadHook.java

Violation	Line
Avoid modifiers which are implied by the context	60–65
Avoid modifiers which are implied by the context	75

org/eclipse/jgit/transport/PreReceiveHook.java

Violation	Line
Avoid modifiers which are implied by the context	82–88
Avoid modifiers which are implied by the context	102

org/eclipse/jgit/transport/PreUploadHook.java

Violation	Line
Avoid modifiers which are implied by the context	62–85
Avoid modifiers which are implied by the context	99–101
Avoid modifiers which are implied by the context	123–126
Avoid modifiers which are implied by the context	144–146

org/eclipse/jgit/transport/PushCertificateIdent.java

Violation	Line
Useless parentheses.	267

org/eclipse/jgit/transport/PushCertificateParser.java

Violation	Line
Avoid modifiers which are implied by the context	87–98
Avoid empty catch blocks	265–268

org/eclipse/jgit/transport/PushConnection.java

Violation	Line
Avoid modifiers which are implied by the context	116–118
Avoid modifiers which are implied by the context	166–168

org/eclipse/jgit/transport/PushProcess.java

Violation	Line
Avoid empty catch blocks	296–298

org/eclipse/jgit/transport/ReceiveCommand.java

Violation	Line
Avoid modifiers which are implied by the context	75–98
Avoid modifiers which are implied by the context	101–134
Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED'	279
Useless parentheses.	777
These nested if statements could be combined	780–784
Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED'	843
Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON'	871
Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON'	875
Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON'	881

org/eclipse/jgit/transport/ReceivePack.java

Violation	Line
Avoid empty catch blocks	323–325

org/eclipse/jgit/transport/RefAdvertiser.java

Violation	Line
Unnecessary use of fully qualified name 'Constants.OBJECT_ID_STRING_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_STRING_LENGTH'	167
Useless parentheses.	295

org/eclipse/jgit/transport/RefFilter.java

Violation	Line
Avoid modifiers which are implied by the context	64–69
Avoid modifiers which are implied by the context	79

org/eclipse/jgit/transport/RefSpec.java

Violation	Line
These nested if statements could be combined	553–554

org/eclipse/jgit/transport/RemoteRefUpdate.java

Violation	Line
Avoid modifiers which are implied by the context	75–127

org/eclipse/jgit/transport/RemoteSession.java

Violation	Line
Avoid modifiers which are implied by the context	81
Avoid modifiers which are implied by the context	86

org/eclipse/jgit/transport/SideBandInputStream.java

Violation	Line
Useless parentheses.	154

org/eclipse/jgit/transport/Transport.java

Violation	Line
Avoid empty catch blocks	170–173
Avoid empty catch blocks	176–178

org/eclipse/jgit/transport/TransportAmazonS3.java

Violation	Line
Avoid empty catch blocks	263–265

org/eclipse/jgit/transport/TransportBundle.java

Violation	Line
Avoid modifiers which are implied by the context	61

org/eclipse/jgit/transport/TransportBundleFile.java

Violation	Line
Useless parentheses.	89

org/eclipse/jgit/transport/TransportBundleStream.java

Violation	Line
Avoid empty catch blocks	118–120

org/eclipse/jgit/transport/TransportGitAnon.java

Violation	Line
Avoid empty catch blocks	153–155
Avoid empty catch blocks	213–215
Avoid empty catch blocks	252–254

org/eclipse/jgit/transport/TransportGitSsh.java

Violation	Line
Useless parentheses.	182
Avoid empty catch blocks	352–354
Avoid empty catch blocks	359–361

org/eclipse/jgit/transport/TransportHttp.java

Violation	Line
Too many static imports may lead to messy code	47–1454
These nested if statements could be combined	760–762
Avoid empty while statements	910–912
Avoid empty catch blocks	931–933
Avoid empty catch blocks	937–939

org/eclipse/jgit/transport/TransportLocal.java

Violation	Line
Useless parentheses.	115

org/eclipse/jgit/transport/TransportProtocol.java

Violation	Line
Avoid modifiers which are implied by the context	93–104

org/eclipse/jgit/transport/TransportSftp.java

Violation	Line
Avoid empty catch blocks	362–365

org/eclipse/jgit/transport/URIish.java

Violation	Line
Useless parentheses.	275–276
Useless parentheses.	346
Useless parentheses.	347

org/eclipse/jgit/transport/UploadPack.java

Violation	Line
Too many static imports may lead to messy code	44–1727
Avoid modifiers which are implied by the context	114–144
Useless parentheses.	677–678
Avoid empty catch blocks	834–836
Avoid empty catch blocks	847–849
Avoid empty while statements	856–858
Avoid empty catch blocks	891–893
Avoid empty catch blocks	1005–1007

org/eclipse/jgit/transport/UploadPackLogger.java

Violation	Line
Avoid modifiers which are implied by the context	62–67
Avoid modifiers which are implied by the context	76

org/eclipse/jgit/transport/UsernamePasswordCredentialsProvider.java

Violation	Line
These nested if statements could be combined	120–124

org/eclipse/jgit/transport/WalkFetchConnection.java

Violation	Line
Avoid using a branching statement as the last in a loop.	597
Avoid empty catch blocks	846–848

org/eclipse/jgit/transport/WalkPushConnection.java

Violation	Line
Avoid empty catch blocks	302–306

org/eclipse/jgit/transport/WalkRemoteObjectDatabase.java

Violation	Line
Avoid empty catch blocks	421–423

org/eclipse/jgit/transport/http/HttpConnection.java

Violation	Line
Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_OK' due to existing import 'java.net.HttpURLConnection'	72
Avoid modifiers which are implied by the context	72
Avoid modifiers which are implied by the context	78
Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_PERM' due to existing import 'java.net.HttpURLConnection'	78
Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_TEMP' due to existing import 'java.net.HttpURLConnection'	84
Avoid modifiers which are implied by the context	84
Avoid modifiers which are implied by the context	90
Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_SEE_OTHER' due to existing import 'java.net.HttpURLConnection'	90
Avoid modifiers which are implied by the context	98
Avoid modifiers which are implied by the context	103
Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_NOT_FOUND' due to existing import 'java.net.HttpURLConnection'	103
Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_UNAUTHORIZED' due to existing import 'java.net.HttpURLConnection'	108
Avoid modifiers which are implied by the context	108
Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_FORBIDDEN' due to existing import 'java.net.HttpURLConnection'	113
Avoid modifiers which are implied by the context	113
Avoid modifiers which are implied by the context	122
Avoid modifiers which are implied by the context	130
Avoid modifiers which are implied by the context	139
Avoid modifiers which are implied by the context	147
Avoid modifiers which are implied by the context	159
Avoid modifiers which are implied by the context	173–174
Avoid modifiers which are implied by the context	184
Avoid modifiers which are implied by the context	194
Avoid modifiers which are implied by the context	204
Avoid modifiers which are implied by the context	213
Avoid modifiers which are implied by the context	225
Avoid modifiers which are implied by the context	236
Avoid modifiers which are implied by the context	246
Avoid modifiers which are implied by the context	256
Avoid modifiers which are implied by the context	265
Avoid modifiers which are implied by the context	274
Avoid modifiers which are implied by the context	283
Avoid modifiers which are implied by the context	293
Avoid modifiers which are implied by the context	301
Avoid modifiers which are implied by the context	309
Avoid modifiers which are implied by the context	317
Avoid modifiers which are implied by the context	335–337
Avoid modifiers which are implied by the context	348–349

org/eclipse/jgit/transport/http/HttpConnectionFactory.java

Violation	Line
Avoid modifiers which are implied by the context	65
Avoid modifiers which are implied by the context	78–79

org/eclipse/jgit/transport/resolver/ReceivePackFactory.java

Violation	Line
Avoid modifiers which are implied by the context	60–66

org/eclipse/jgit/transport/resolver/RepositoryResolver.java

Violation	Line
Avoid modifiers which are implied by the context	60–66

org/eclipse/jgit/transport/resolver/UploadPackFactory.java

Violation	Line
Avoid modifiers which are implied by the context	60–66

org/eclipse/jgit/treewalk/CanonicalTreeParser.java

Violation	Line
Too many static imports may lead to messy code	45–445
Avoid using a branching statement as the last in a loop.	202

org/eclipse/jgit/treewalk/NameConflictTreeWalk.java

Violation	Line
Avoid using a branching statement as the last in a loop.	149

org/eclipse/jgit/treewalk/TreeWalk.java

Violation	Line
Avoid modifiers which are implied by the context	111–122
Useless parentheses.	648
Avoid using a branching statement as the last in a loop.	877
Useless parentheses.	1315
Useless parentheses.	1455

org/eclipse/jgit/treewalk/WorkingTreeIterator.java

Violation	Line
Useless parentheses.	282–283
Avoid empty catch blocks	436–440
Avoid empty while statements	476–478
These nested if statements could be combined	852–853
Useless parentheses.	1471–1472
Useless parentheses.	1473–1474

org/eclipse/jgit/treewalk/filter/ByteArraySet.java

Violation	Line
Useless parentheses.	77

org/eclipse/jgit/treewalk/filter/IndexDiffFilter.java

Violation	Line
Avoid unused imports such as 'org.eclipse.jgit.lib.ObjectReader'	56

org/eclipse/jgit/treewalk/filter/TreeFilterMarker.java

Violation	Line
Useless parentheses.	116

org/eclipse/jgit/util/Base64.java

Violation	Line
Useless parentheses.	121
Useless parentheses.	128
Useless parentheses.	135
Unnecessary use of fully qualified name 'StandardCharsets.UTF_8' due to existing static import 'java.nio.charset.StandardCharsets.UTF_8'	181
Useless parentheses.	230

org/eclipse/jgit/util/ChangeIdUtil.java

Violation	Line
Avoid using a branching statement as the last in a loop.	200
Avoid using a branching statement as the last in a loop.	293

org/eclipse/jgit/util/FS.java

Violation	Line
Avoid empty if statements	624–627
Useless parentheses.	695
Avoid empty catch blocks	1205–1212
Avoid empty catch blocks	1533–1535

org/eclipse/jgit/util/FS_POSIX.java

Violation	Line
Do not start a literal by 0 unless its an octal value	83
These nested if statements could be combined	199–217
These nested if statements could be combined	200–216
Useless parentheses.	473
Useless parentheses.	473

org/eclipse/jgit/util/FS_Win32.java

Violation	Line
These nested if statements could be combined	129–145

org/eclipse/jgit/util/FileUtils.java

Violation	Line
Avoid empty catch blocks	295–297
Avoid using a branching statement as the last in a loop.	496
Ternary operators that can be simplified with \|\| or &&	809

org/eclipse/jgit/util/GSSManagerFactory.java

Violation	Line
Useless parentheses.	64

org/eclipse/jgit/util/GitDateFormatter.java

Violation	Line
Avoid modifiers which are implied by the context	71–117

org/eclipse/jgit/util/GitDateParser.java

Violation	Line
Avoid empty catch blocks	231–233

org/eclipse/jgit/util/IO.java

Violation	Line
Avoid empty catch blocks	119–121
Avoid empty catch blocks	179–181

org/eclipse/jgit/util/LfsFactory.java

Violation	Line
Avoid modifiers which are implied by the context	315

org/eclipse/jgit/util/NB.java

Violation	Line
Useless parentheses.	91
Useless parentheses.	203
Useless parentheses.	206

org/eclipse/jgit/util/ProcessResult.java

Violation	Line
Avoid modifiers which are implied by the context	54–69

org/eclipse/jgit/util/RawParseUtils.java

Violation	Line
Too many static imports may lead to messy code	45–1255
Unnecessary final modifier in final class	128–135
Unnecessary final modifier in final class	209–243
Useless parentheses.	234
Avoid empty catch blocks	237–239
Unnecessary final modifier in final class	263–297
Useless parentheses.	288
Avoid empty catch blocks	291–293
Unnecessary final modifier in final class	314–327
Unnecessary final modifier in final class	344–368
Unnecessary final modifier in final class	386–434
Unnecessary final modifier in final class	445–450
Unnecessary final modifier in final class	464–466
Unnecessary final modifier in final class	484–490
Unnecessary final modifier in final class	503–510
Unnecessary final modifier in final class	523–525
Unnecessary final modifier in final class	540–548
Unnecessary final modifier in final class	561–569
Unnecessary final modifier in final class	582–584
Unnecessary final modifier in final class	600–609
Unnecessary final modifier in final class	637–663
Useless parentheses.	659
Unnecessary final modifier in final class	678–685
Unnecessary final modifier in final class	700–709
Unnecessary final modifier in final class	724–737
Unnecessary final modifier in final class	752–762
Useless parentheses.	869
Unnecessary final modifier in final class	1165–1176
Unnecessary final modifier in final class	1189–1198
Unnecessary final modifier in final class	1214–1224
Useless parentheses.	1217

org/eclipse/jgit/util/RawSubStringPattern.java

Violation	Line
Avoid empty while statements	100–102
Avoid using a branching statement as the last in a loop.	112

org/eclipse/jgit/util/RefMap.java

Violation	Line
Invoke equals() on the object you've already ensured is not null	428

org/eclipse/jgit/util/RelativeDateFormatter.java

Violation	Line
Useless parentheses.	122
Useless parentheses.	126
Useless parentheses.	127

org/eclipse/jgit/util/TemporaryBuffer.java

Violation	Line
Avoid empty catch blocks	405–407

org/eclipse/jgit/util/io/StreamCopyThread.java

Violation	Line
Avoid empty catch blocks	163–165
Avoid empty catch blocks	168–170

org/eclipse/jgit/util/sha1/SHA1.java

Violation	Line
Ternary operators that can be simplified with \|\| or &&	85
Unnecessary final modifier in final class	589–592
Unnecessary final modifier in final class	594–600

org/eclipse/jgit/util/time/MonotonicClock.java

Violation	Line
Avoid unused imports such as 'java.time.Duration'	46