The following document contains the results of PMD 6.13.0.
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 181 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 619 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 500 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 593 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 639 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 356 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 535 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 225 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 540 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 603 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 707 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 388 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 136 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 626 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 594 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 202 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 149 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 864 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 200 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 293 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 525 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 112 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Format': member interfaces are implicitly static | 127–194 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Stage': nested enums are implicitly static | 140–161 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Stage(int)': enum constructors are implicitly private | 158–160 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 719–722 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 285–290 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 723–728 |
CollapsibleIfStatements | These nested if statements could be combined | 1306–1309 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 405–409 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused method parameters such as 'state'. | 216 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'State': nested enums are implicitly static | 70–86 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 157–159 |
Rule | Violation | Line |
---|---|---|
EmptyWhileStmt | Avoid empty while statements | 431–433 |
EmptyIfStmt | Avoid empty if statements | 521–525 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isFile': private methods cannot be overridden | 1019–1021 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'RenameDetectionType': nested enums are implicitly static | 62–71 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'ChangeType': nested enums are implicitly static | 75–90 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Side': nested enums are implicitly static | 93–99 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 45–1243 |
EmptyCatchBlock | Avoid empty catch blocks | 722–724 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static | 67–79 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'fs'. | 369 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 459–462 |
EmptyIfStmt | Avoid empty if statements | 464–469 |
CollapsibleIfStatements | These nested if statements could be combined | 921–930 |
CollapsibleIfStatements | These nested if statements could be combined | 924–929 |
CollapsibleIfStatements | These nested if statements could be combined | 1238–1240 |
EmptyWhileStmt | Avoid empty while statements | 1615–1617 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 248–254 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'MergeBaseFailureReason': nested enums are implicitly static | 66–85 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'otp'. | 63 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 63–65 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 135–142 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 164–166 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 173–175 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 182–184 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 191–193 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 200–202 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 209–211 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 222–224 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 57–59 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 54–56 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readIncludeFile': the method is declared in an interface type | 111–112 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'sha1': the method is declared in an interface type | 157 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readFile': the method is declared in an interface type | 177–180 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readFileWithMode': the method is declared in an interface type | 204–205 |
EmptyCatchBlock | Avoid empty catch blocks | 592–594 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 162–164 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'MatchResult': nested enums are implicitly static | 62–79 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'public static final' on field 'NO_MATCH': the field is declared in an interface type | 53–64 |
Rule | Violation | Line |
---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 66 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 43–343 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'PatternState': nested enums are implicitly static | 234–236 |
CollapsibleIfStatements | These nested if statements could be combined | 337–339 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 64–68 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 72–74 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 106–108 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–637 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'State': nested enums are implicitly static | 129–141 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–158 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–791 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–374 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–185 |
EmptyCatchBlock | Avoid empty catch blocks | 150–152 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–237 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–448 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'State(boolean)': enum constructors are implicitly private | 114–116 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–326 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–247 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 81–83 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 89–91 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–277 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–788 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 248–250 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'PackSource': nested enums are implicitly static | 100–235 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–529 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isGarbage': the method is already in a final class | 249–251 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 183–185 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 359–361 |
EmptyCatchBlock | Avoid empty catch blocks | 379–381 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–182 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 377–379 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 176–178 |
Rule | Violation | Line |
---|---|---|
OverrideBothEqualsAndHashcode | Ensure you override both equals() and hashCode() | 141 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–460 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 122–124 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'clear': the method is already in a final class | 62–64 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'set': the method is already in a final class | 66–74 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'clear': the method is already in a final class | 76–80 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'get': the method is already in a final class | 82–85 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'toEWAHCompressedBitmap': the method is already in a final class | 87–112 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'block': private methods cannot be overridden | 114–116 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'mask': private methods cannot be overridden | 118–120 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofObjectType': private methods cannot be overridden | 344–346 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 112–114 |
CollapsibleIfStatements | These nested if statements could be combined | 199–201 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'InsertLooseObjectResult': nested enums are implicitly static | 63–65 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 218–221 |
EmptyCatchBlock | Avoid empty catch blocks | 291–293 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 422–424 |
EmptyCatchBlock | Avoid empty catch blocks | 459–461 |
EmptyCatchBlock | Avoid empty catch blocks | 577–580 |
EmptyCatchBlock | Avoid empty catch blocks | 959–961 |
EmptyCatchBlock | Avoid empty catch blocks | 984–986 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 111–113 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'maybeContains': the method is already in a final class | 76–80 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'contains': the method is already in a final class | 82–124 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'get': private methods cannot be overridden | 126–129 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'block': private methods cannot be overridden | 131–133 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'mask': private methods cannot be overridden | 135–137 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isEmpty': private methods cannot be overridden | 139–141 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'or': the method is already in a final class | 143–147 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'andNot': the method is already in a final class | 149–153 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'xor': the method is already in a final class | 155–162 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getBitmap': the method is already in a final class | 164–166 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 98–100 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 326–328 |
CollapsibleIfStatements | These nested if statements could be combined | 342–344 |
EmptyCatchBlock | Avoid empty catch blocks | 729–731 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 220–222 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'decompress': private methods cannot be overridden | 398–419 |
EmptyCatchBlock | Avoid empty catch blocks | 734–738 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 349–351 |
EmptyCatchBlock | Avoid empty catch blocks | 356–358 |
EmptyCatchBlock | Avoid empty catch blocks | 361–363 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'fs'. | 68 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–551 |
EmptyCatchBlock | Avoid empty catch blocks | 451–453 |
EmptyCatchBlock | Avoid empty catch blocks | 472–486 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 47–1511 |
CollapsibleIfStatements | These nested if statements could be combined | 557–558 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'LooseRef': member interfaces are implicitly static | 1405–1409 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 217–219 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 46–274 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 69–73 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'bits': private methods cannot be overridden | 127–133 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 136–138 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on class 'PackInfo': members of interface types are implicitly public | 75–102 |
UnnecessaryModifier | Unnecessary modifier 'static' on class 'PackInfo': types nested within an interface type are implicitly static | 75–102 |
Rule | Violation | Line |
---|---|---|
ForLoopShouldBeWhileLoop | This for loop could be simplified to a while loop | 139–149 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'size': the method is already in a final class | 262–264 |
SimplifiedTernary | Ternary operators that can be simplified with || or && | 350 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 309–311 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'set': the method is already in a final class | 57–61 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'depth': the method is already in a final class | 64–66 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'type': the method is already in a final class | 69–71 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'size': the method is already in a final class | 74–76 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'empty': the method is already in a final class | 79–81 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'makeNext': the method is already in a final class | 83–93 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'write': the method is already in a final class | 110–114 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'write': the method is already in a final class | 118–138 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeFileHeader': the method is already in a final class | 146–153 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeObject': the method is already in a final class | 170–172 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeHeader': the method is already in a final class | 191–206 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'objectHeader': private methods cannot be overridden | 208–217 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofsDelta': private methods cannot be overridden | 219–226 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofsDeltaVarIntLength': private methods cannot be overridden | 228–233 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getCopyBuffer': the method is already in a final class | 240–242 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'length': the method is already in a final class | 253–255 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getDigest': the method is already in a final class | 258–260 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 45–2518 |
EmptyWhileStmt | Avoid empty while statements | 1949–1951 |
CollapsibleIfStatements | These nested if statements could be combined | 2289–2290 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'have': private methods cannot be overridden | 2304–2307 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'PackingPhase': nested enums are implicitly static | 2433–2451 |
Rule | Violation | Line |
---|---|---|
EmptyWhileStmt | Avoid empty while statements | 291–293 |
CollapsibleIfStatements | These nested if statements could be combined | 303–307 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–600 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–605 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–249 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–727 |
UnusedNullCheckInEquals | Invoke equals() on the object you've already ensured is not null | 710 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–859 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–342 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–417 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–227 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–287 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 43–206 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 372–374 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 234–236 |
EmptyCatchBlock | Avoid empty catch blocks | 446–448 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isId': the method is already in a final class | 77–87 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromString': the method is already in a final class | 101–109 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromObjectId': the method is already in a final class | 122–125 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromString': the method is already in a final class | 134–139 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromHexString': private methods cannot be overridden | 141–153 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'hexUInt32': private methods cannot be overridden | 155–166 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 247–267 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 283–303 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 319–339 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getFirstByte': the method is already in a final class | 346–348 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'name': the method is already in a final class | 376–397 |
Rule | Violation | Line |
---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 299 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–783 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on interface 'Bitmap': members of interface types are implicitly public | 77–117 |
UnnecessaryModifier | Unnecessary modifier 'public' on interface 'BitmapBuilder': members of interface types are implicitly public | 123–211 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'BranchRebaseMode(String)': enum constructors are implicitly private | 75–77 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'SectionParser': member interfaces are implicitly static | 1476–1485 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ConfigEnum': member interfaces are implicitly static | 1515–1531 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'AutoCRLF': nested enums are implicitly static | 63–72 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'EOL': nested enums are implicitly static | 81–90 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'EolStreamType': nested enums are implicitly static | 97–112 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'CheckStat': nested enums are implicitly static | 119–131 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'SymLinks': nested enums are implicitly static | 148–153 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'HideDotFiles': nested enums are implicitly static | 160–167 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 128–130 |
Rule | Violation | Line |
---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 68 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 71 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 74 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 77 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 80 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 83 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 108 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 113 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 118 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 123 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 163 |
Rule | Violation | Line |
---|---|---|
ImportFromSamePackage | No need to import a type that lives in the same package | 45 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'GpgFormat(String)': enum constructors are implicitly private | 64–66 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'StageState': nested enums are implicitly static | 111–199 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'StageState(int)': enum constructors are implicitly private | 149–151 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getWorkingTreeIterator': the method is declared in an interface type | 352 |
CollapsibleIfStatements | These nested if statements could be combined | 474–477 |
CollapsibleIfStatements | These nested if statements could be combined | 519–523 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 45–1277 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromHexString': private methods cannot be overridden | 241–253 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'newSegment': private methods cannot be overridden | 340–342 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'computeGrowAt': private methods cannot be overridden | 344–346 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'equals': private methods cannot be overridden | 348–355 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'createArray': private methods cannot be overridden | 258–260 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Action': nested enums are implicitly static | 61–118 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Action(String, String)': enum constructors are implicitly private | 87–90 |
CollapsibleIfStatements | These nested if statements could be combined | 191–194 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Storage(boolean, boolean)': enum constructors are implicitly private | 108–111 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Result': nested enums are implicitly static | 69–172 |
CollapsibleIfStatements | These nested if statements could be combined | 782–784 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 898–899 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Key': member interfaces are implicitly static | 344–365 |
Rule | Violation | Line |
---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 95–97 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'FetchRecurseSubmodulesMode(String)': enum constructors are implicitly private | 74–76 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–403 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 143–145 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 72–74 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 47–1355 |
CollapsibleIfStatements | These nested if statements could be combined | 408–410 |
CollapsibleIfStatements | These nested if statements could be combined | 823–849 |
CollapsibleIfStatements | These nested if statements could be combined | 839–841 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'inCore'. | 86 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static | 61–67 |
Rule | Violation | Line |
---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 152–154 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–653 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'PatchType': nested enums are implicitly static | 107–116 |
EmptyCatchBlock | Avoid empty catch blocks | 248–251 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Severity': nested enums are implicitly static | 57–63 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–392 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 198–200 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 138–141 |
EmptyCatchBlock | Avoid empty catch blocks | 157–172 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on class 'Commit': members of interface types are implicitly public | 109–145 |
UnnecessaryModifier | Unnecessary modifier 'static' on class 'Commit': types nested within an interface type are implicitly static | 109–145 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'RevWalk': members of interface types are implicitly public | 148–275 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'ObjectWalk': members of interface types are implicitly public | 278–393 |
Rule | Violation | Line |
---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 87 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 88 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 89 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 90 |
EmptyWhileStmt | Avoid empty while statements | 818–820 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 193–196 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 343–345 |
EmptyCatchBlock | Avoid empty catch blocks | 354–356 |
EmptyCatchBlock | Avoid empty catch blocks | 384–386 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 431–433 |
EmptyCatchBlock | Avoid empty catch blocks | 487–489 |
EmptyCatchBlock | Avoid empty catch blocks | 507–510 |
EmptyCatchBlock | Avoid empty catch blocks | 788–790 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 185–187 |
EmptyCatchBlock | Avoid empty catch blocks | 191–193 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–1971 |
EmptyCatchBlock | Avoid empty catch blocks | 380–382 |
EmptyCatchBlock | Avoid empty catch blocks | 391–393 |
EmptyCatchBlock | Avoid empty catch blocks | 407–409 |
EmptyCatchBlock | Avoid empty catch blocks | 1040–1042 |
EmptyCatchBlock | Avoid empty catch blocks | 1347–1349 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 264–266 |
EmptyCatchBlock | Avoid empty catch blocks | 301–303 |
Rule | Violation | Line |
---|---|---|
AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 114 |
EmptyCatchBlock | Avoid empty catch blocks | 315–317 |
EmptyCatchBlock | Avoid empty catch blocks | 341–343 |
EmptyCatchBlock | Avoid empty catch blocks | 425–427 |
EmptyCatchBlock | Avoid empty catch blocks | 427–429 |
EmptyCatchBlock | Avoid empty catch blocks | 429–431 |
EmptyCatchBlock | Avoid empty catch blocks | 434–436 |
EmptyCatchBlock | Avoid empty catch blocks | 439–441 |
Rule | Violation | Line |
---|---|---|
UselessQualifiedThis | Useless qualified this usage in the same class. | 72 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 45–561 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 95–98 |
EmptyCatchBlock | Avoid empty catch blocks | 107–110 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on class 'FtpException': types nested within an interface type are implicitly static | 64–97 |
EmptyCatchBlock | Avoid empty catch blocks | 221–223 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'MultiAck': nested enums are implicitly static | 283–285 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 192–194 |
EmptyIfStmt | Avoid empty if statements | 416–418 |
EmptyIfStmt | Avoid empty if statements | 419–421 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'HttpRedirectMode(String)': enum constructors are implicitly private | 156–158 |
EmptyIfStmt | Avoid empty if statements | 465–467 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 110–113 |
EmptyCatchBlock | Avoid empty catch blocks | 119–121 |
EmptyCatchBlock | Avoid empty catch blocks | 125–127 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 103–106 |
EmptyCatchBlock | Avoid empty catch blocks | 115–117 |
EmptyCatchBlock | Avoid empty catch blocks | 121–123 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 432–434 |
EmptyCatchBlock | Avoid empty catch blocks | 434–436 |
EmptyCatchBlock | Avoid empty catch blocks | 455–457 |
CollapsibleIfStatements | These nested if statements could be combined | 519–532 |
EmptyCatchBlock | Avoid empty catch blocks | 529–531 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 190–192 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 103–107 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Source': nested enums are implicitly static | 106–112 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'checkIfTooLarge': private methods cannot be overridden | 731–754 |
EmptyCatchBlock | Avoid empty catch blocks | 1184–1188 |
EmptyCatchBlock | Avoid empty catch blocks | 1200–1204 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'AckNackResult': nested enums are implicitly static | 98–109 |
EmptyCatchBlock | Avoid empty catch blocks | 335–337 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 143–148 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on field 'DEFAULT': the field is declared in an interface type | 55–57 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 43–281 |
EmptyIfStmt | Avoid empty if statements | 102–104 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'StringReader': member interfaces are implicitly static | 87–98 |
EmptyCatchBlock | Avoid empty catch blocks | 265–268 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'PushRecurseSubmodulesMode(String)': enum constructors are implicitly private | 76–78 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 296–298 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static | 75–98 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Result': nested enums are implicitly static | 101–134 |
CollapsibleIfStatements | These nested if statements could be combined | 780–784 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 421–423 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 554–555 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Status': nested enums are implicitly static | 75–127 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'TagOpt(String)': enum constructors are implicitly private | 87–89 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 162–164 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 263–265 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 118–120 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 154–156 |
EmptyCatchBlock | Avoid empty catch blocks | 214–216 |
EmptyCatchBlock | Avoid empty catch blocks | 253–255 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 353–355 |
EmptyCatchBlock | Avoid empty catch blocks | 360–362 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 47–1699 |
CollapsibleIfStatements | These nested if statements could be combined | 855–857 |
EmptyWhileStmt | Avoid empty while statements | 1155–1157 |
EmptyCatchBlock | Avoid empty catch blocks | 1176–1178 |
EmptyCatchBlock | Avoid empty catch blocks | 1182–1184 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'URIishField': nested enums are implicitly static | 93–104 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 346–349 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 44–2414 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'RequestPolicy': nested enums are implicitly static | 137–167 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'IOConsumer': member interfaces are implicitly static | 236–238 |
EmptyWhileStmt | Avoid empty while statements | 1072–1074 |
EmptyWhileStmt | Avoid empty while statements | 1321–1323 |
EmptyWhileStmt | Avoid empty while statements | 1330–1332 |
EmptyCatchBlock | Avoid empty catch blocks | 1377–1379 |
EmptyCatchBlock | Avoid empty catch blocks | 1545–1547 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 120–124 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 843–845 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 302–306 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 421–423 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 45–445 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'OperationType': nested enums are implicitly static | 114–125 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 471–475 |
EmptyWhileStmt | Avoid empty while statements | 511–513 |
CollapsibleIfStatements | These nested if statements could be combined | 895–896 |
EmptyCatchBlock | Avoid empty catch blocks | 1577–1579 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 388–390 |
EmptyIfStmt | Avoid empty if statements | 1258–1261 |
EmptyCatchBlock | Avoid empty catch blocks | 1875–1882 |
EmptyCatchBlock | Avoid empty catch blocks | 2214–2216 |
Rule | Violation | Line |
---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 91 |
CollapsibleIfStatements | These nested if statements could be combined | 176–194 |
CollapsibleIfStatements | These nested if statements could be combined | 177–193 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 164–166 |
CollapsibleIfStatements | These nested if statements could be combined | 180–196 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 324–326 |
SimplifiedTernary | Ternary operators that can be simplified with || or && | 846 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Format': nested enums are implicitly static | 71–117 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'ParseableSimpleDateFormat(String)': enum constructors are implicitly private | 131–133 |
EmptyCatchBlock | Avoid empty catch blocks | 231–233 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'setRepository': the method is declared in an interface type | 322 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'index': private methods cannot be overridden | 169–173 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'createArray': private methods cannot be overridden | 176–178 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Status': nested enums are implicitly static | 54–69 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 45–1340 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'match': the method is already in a final class | 131–138 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseBase10': the method is already in a final class | 212–246 |
EmptyCatchBlock | Avoid empty catch blocks | 240–242 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseLongBase10': the method is already in a final class | 266–300 |
EmptyCatchBlock | Avoid empty catch blocks | 294–296 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt16': the method is already in a final class | 317–330 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt32': the method is already in a final class | 347–371 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt64': the method is already in a final class | 389–437 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt4': the method is already in a final class | 448–453 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseTimeZoneOffset': the method is already in a final class | 467–469 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseTimeZoneOffset': the method is already in a final class | 487–493 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'next': the method is already in a final class | 506–513 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'nextLF': the method is already in a final class | 526–528 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'nextLF': the method is already in a final class | 543–551 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'headerEnd': the method is already in a final class | 564–573 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'headerStart': the method is already in a final class | 588–607 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prev': the method is already in a final class | 620–628 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prevLF': the method is already in a final class | 641–643 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prevLF': the method is already in a final class | 659–668 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lineMap': the method is already in a final class | 692–700 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lineMapOrBinary': the method is already in a final class | 718–725 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'author': the method is already in a final class | 763–770 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'committer': the method is already in a final class | 785–794 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'tagger': the method is already in a final class | 809–822 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'encoding': the method is already in a final class | 837–847 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'commitMessage': the method is already in a final class | 1250–1261 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'tagMessage': the method is already in a final class | 1274–1283 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'endOfParagraph': the method is already in a final class | 1299–1309 |
Rule | Violation | Line |
---|---|---|
EmptyWhileStmt | Avoid empty while statements | 100–102 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'neq': private methods cannot be overridden | 117–119 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lc': private methods cannot be overridden | 121–123 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'isSameObject': the method is declared in an interface type | 64–66 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 405–407 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 71–73 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 149–151 |
EmptyCatchBlock | Avoid empty catch blocks | 154–156 |
Rule | Violation | Line |
---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 87 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'init': the method is already in a final class | 592–595 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'save': the method is already in a final class | 597–603 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 212–213 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'FileMode.GITLINK' due to existing static import 'org.eclipse.jgit.lib.FileMode.GITLINK' | 213 |
UselessParentheses | Useless parentheses. | 214–215 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 582 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 204 |
UselessParentheses | Useless parentheses. | 205 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 419–420 |
UselessParentheses | Useless parentheses. | 421 |
UselessParentheses | Useless parentheses. | 554 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_HEADS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_HEADS' | 256 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 455 |
UselessParentheses | Useless parentheses. | 455 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 195–197 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 416 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 284 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 790–791 |
UselessParentheses | Useless parentheses. | 790 |
UselessParentheses | Useless parentheses. | 791 |
UselessParentheses | Useless parentheses. | 1753 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 159 |
UselessParentheses | Useless parentheses. | 172 |
UselessParentheses | Useless parentheses. | 185 |
UselessParentheses | Useless parentheses. | 199 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 144 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 320 |
UselessParentheses | Useless parentheses. | 442–443 |
UselessParentheses | Useless parentheses. | 501 |
UselessParentheses | Useless parentheses. | 549 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 83 |
UselessParentheses | Useless parentheses. | 133 |
UselessParentheses | Useless parentheses. | 171 |
UselessParentheses | Useless parentheses. | 206 |
UselessParentheses | Useless parentheses. | 252 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 381 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 670 |
UselessParentheses | Useless parentheses. | 748 |
UselessParentheses | Useless parentheses. | 749 |
UselessParentheses | Useless parentheses. | 750 |
UselessParentheses | Useless parentheses. | 751 |
UselessParentheses | Useless parentheses. | 752 |
UselessParentheses | Useless parentheses. | 753 |
UselessParentheses | Useless parentheses. | 812 |
UselessParentheses | Useless parentheses. | 813 |
UselessParentheses | Useless parentheses. | 813 |
UselessParentheses | Useless parentheses. | 813 |
UselessParentheses | Useless parentheses. | 909 |
UselessParentheses | Useless parentheses. | 1152–1153 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 125 |
UselessParentheses | Useless parentheses. | 294 |
UselessParentheses | Useless parentheses. | 418 |
UselessParentheses | Useless parentheses. | 418 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 494 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 782 |
UselessParentheses | Useless parentheses. | 783 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 130 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Strings.getPathSeparator' due to existing static import 'org.eclipse.jgit.ignore.internal.Strings.getPathSeparator' | 131 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 138 |
UselessParentheses | Useless parentheses. | 398 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 67 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 427 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 477 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 626 |
UselessParentheses | Useless parentheses. | 633 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 205 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 579 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 541 |
UselessParentheses | Useless parentheses. | 547 |
UselessParentheses | Useless parentheses. | 554 |
UselessParentheses | Useless parentheses. | 560 |
UselessParentheses | Useless parentheses. | 577 |
UselessParentheses | Useless parentheses. | 601 |
UselessParentheses | Useless parentheses. | 687 |
UselessParentheses | Useless parentheses. | 695 |
UselessParentheses | Useless parentheses. | 768 |
UselessParentheses | Useless parentheses. | 818 |
UselessParentheses | Useless parentheses. | 937 |
UselessParentheses | Useless parentheses. | 1047 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 762 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 240–241 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 579 |
UselessParentheses | Useless parentheses. | 579 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 979–981 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX' | 1157 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX' | 1160 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 654 |
UselessParentheses | Useless parentheses. | 655 |
UselessParentheses | Useless parentheses. | 775 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 163 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 465 |
UselessParentheses | Useless parentheses. | 471 |
UselessParentheses | Useless parentheses. | 478 |
UselessParentheses | Useless parentheses. | 484 |
UselessParentheses | Useless parentheses. | 501 |
UselessParentheses | Useless parentheses. | 534 |
UselessParentheses | Useless parentheses. | 554 |
UselessParentheses | Useless parentheses. | 611 |
UselessParentheses | Useless parentheses. | 619 |
UselessParentheses | Useless parentheses. | 869 |
UselessParentheses | Useless parentheses. | 918 |
UselessParentheses | Useless parentheses. | 1033 |
UselessParentheses | Useless parentheses. | 1124 |
UselessParentheses | Useless parentheses. | 1139 |
UselessParentheses | Useless parentheses. | 1225 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 240 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 231 |
UselessParentheses | Useless parentheses. | 338 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 393 |
UselessParentheses | Useless parentheses. | 394 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 546 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1183 |
UselessParentheses | Useless parentheses. | 1198 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.HEAD' due to existing static import 'org.eclipse.jgit.lib.Constants.HEAD' | 1235 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_REFS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_REFS' | 1266 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 242 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 319 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 68 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 194 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 387 |
UselessParentheses | Useless parentheses. | 2305 |
UselessParentheses | Useless parentheses. | 2306 |
UselessParentheses | Useless parentheses. | 2390 |
UselessParentheses | Useless parentheses. | 2407 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 133 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE' | 234 |
UselessParentheses | Useless parentheses. | 268 |
UselessParentheses | Useless parentheses. | 269–270 |
UselessParentheses | Useless parentheses. | 270 |
UselessParentheses | Useless parentheses. | 271 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE' | 282 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 572 |
UselessParentheses | Useless parentheses. | 584 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 138 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 172 |
UselessParentheses | Useless parentheses. | 482 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 237–238 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 113 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1443 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'StringUtils.toLowerCase' due to existing static import 'org.eclipse.jgit.util.StringUtils.toLowerCase' | 281 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 521–522 |
UselessParentheses | Useless parentheses. | 638–639 |
UselessParentheses | Useless parentheses. | 638 |
UselessParentheses | Useless parentheses. | 638–639 |
UselessParentheses | Useless parentheses. | 767–768 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH' | 571 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH' | 603 |
UselessParentheses | Useless parentheses. | 876 |
UselessParentheses | Useless parentheses. | 936 |
UselessParentheses | Useless parentheses. | 937 |
UselessParentheses | Useless parentheses. | 1015 |
UselessParentheses | Useless parentheses. | 1027 |
UselessParentheses | Useless parentheses. | 1108–1109 |
UselessParentheses | Useless parentheses. | 1110 |
UselessParentheses | Useless parentheses. | 1111 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 631 |
UselessParentheses | Useless parentheses. | 643 |
UselessParentheses | Useless parentheses. | 840 |
UselessParentheses | Useless parentheses. | 841 |
UselessParentheses | Useless parentheses. | 842 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'java.io.File' | 46 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 299 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 86 |
UselessParentheses | Useless parentheses. | 94 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 322 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 90 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 141 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 823 |
UselessParentheses | Useless parentheses. | 823–824 |
UselessParentheses | Useless parentheses. | 823–824 |
UselessParentheses | Useless parentheses. | 887–888 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException' | 51 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException' | 52 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException' | 49 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException' | 50 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 284 |
UselessParentheses | Useless parentheses. | 286 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'RawParseUtils.decode' due to existing static import 'org.eclipse.jgit.util.RawParseUtils.decode' | 297 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 195 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 124 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.jgit.revwalk.RevWalk' due to existing same package import 'org.eclipse.jgit.revwalk.*' | 148 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.jgit.revwalk.ObjectWalk' due to existing same package import 'org.eclipse.jgit.revwalk.*' | 278 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.lib.AnyObjectId' | 61 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 371 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 532 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 680 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 400–401 |
UselessParentheses | Useless parentheses. | 436 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.OK' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.OK' | 556 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'java.util.Collection' | 50 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 175 |
UselessParentheses | Useless parentheses. | 176 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 119–137 |
UselessParentheses | Useless parentheses. | 398–399 |
UselessParentheses | Useless parentheses. | 445 |
UselessParentheses | Useless parentheses. | 465 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'java.io.OutputStream' | 46 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.lib.ProgressMonitor' | 48 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 851 |
UselessParentheses | Useless parentheses. | 1026 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 267 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 279 |
UselessParentheses | Useless parentheses. | 777 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 843 |
UnnecessaryFullyQualifiedName | 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 |
UnnecessaryFullyQualifiedName | 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 |
UnnecessaryFullyQualifiedName | 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 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | 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 |
UselessParentheses | Useless parentheses. | 323 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 355–356 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 154 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 89 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 182 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 115 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 277–278 |
UselessParentheses | Useless parentheses. | 338 |
UselessParentheses | Useless parentheses. | 339 |
UselessParentheses | Useless parentheses. | 366 |
UselessParentheses | Useless parentheses. | 367 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1390 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_OK' due to existing import 'java.net.HttpURLConnection' | 74 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_PERM' due to existing import 'java.net.HttpURLConnection' | 80 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_TEMP' due to existing import 'java.net.HttpURLConnection' | 86 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_SEE_OTHER' due to existing import 'java.net.HttpURLConnection' | 92 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_NOT_FOUND' due to existing import 'java.net.HttpURLConnection' | 105 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_UNAUTHORIZED' due to existing import 'java.net.HttpURLConnection' | 110 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_FORBIDDEN' due to existing import 'java.net.HttpURLConnection' | 115 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1302 |
UselessParentheses | Useless parentheses. | 1439 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 325–326 |
UselessParentheses | Useless parentheses. | 942 |
UselessParentheses | Useless parentheses. | 943 |
UselessParentheses | Useless parentheses. | 946 |
UselessParentheses | Useless parentheses. | 1594–1595 |
UselessParentheses | Useless parentheses. | 1596–1597 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 77 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.lib.ObjectReader' | 56 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 116 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 120 |
UselessParentheses | Useless parentheses. | 127 |
UselessParentheses | Useless parentheses. | 134 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1332 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 485 |
UselessParentheses | Useless parentheses. | 485 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 64 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 261–262 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 91 |
UselessParentheses | Useless parentheses. | 203 |
UselessParentheses | Useless parentheses. | 206 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 237 |
UselessParentheses | Useless parentheses. | 291 |
UselessParentheses | Useless parentheses. | 744 |
UselessParentheses | Useless parentheses. | 954 |
UselessParentheses | Useless parentheses. | 1302 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 122 |
UselessParentheses | Useless parentheses. | 126 |
UselessParentheses | Useless parentheses. | 127 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'java.time.Duration' | 46 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 212–213 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'FileMode.GITLINK' due to existing static import 'org.eclipse.jgit.lib.FileMode.GITLINK' | 4 | 213 |
UselessParentheses | Useless parentheses. | 4 | 214–215 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Format': member interfaces are implicitly static | 3 | 127–194 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Stage': nested enums are implicitly static | 3 | 140–161 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Stage(int)': enum constructors are implicitly private | 3 | 158–160 |
UselessParentheses | Useless parentheses. | 4 | 582 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 204 |
UselessParentheses | Useless parentheses. | 4 | 205 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 719–722 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 419–420 |
UselessParentheses | Useless parentheses. | 4 | 421 |
UselessParentheses | Useless parentheses. | 4 | 554 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_HEADS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_HEADS' | 4 | 256 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 455 |
UselessParentheses | Useless parentheses. | 4 | 455 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 195–197 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 285–290 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 416 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 284 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 723–728 |
UselessParentheses | Useless parentheses. | 4 | 790–791 |
UselessParentheses | Useless parentheses. | 4 | 790 |
UselessParentheses | Useless parentheses. | 4 | 791 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 1306–1309 |
UselessParentheses | Useless parentheses. | 4 | 1753 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 405–409 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused method parameters such as 'state'. | 3 | 216 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'State': nested enums are implicitly static | 3 | 70–86 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 159 |
UselessParentheses | Useless parentheses. | 4 | 172 |
UselessParentheses | Useless parentheses. | 4 | 185 |
UselessParentheses | Useless parentheses. | 4 | 199 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 144 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 157–159 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyWhileStmt | Avoid empty while statements | 3 | 431–433 |
EmptyIfStmt | Avoid empty if statements | 3 | 521–525 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isFile': private methods cannot be overridden | 3 | 1019–1021 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'RenameDetectionType': nested enums are implicitly static | 3 | 62–71 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'ChangeType': nested enums are implicitly static | 3 | 75–90 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Side': nested enums are implicitly static | 3 | 93–99 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 45–1243 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 722–724 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static | 3 | 67–79 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 320 |
UselessParentheses | Useless parentheses. | 4 | 442–443 |
UselessParentheses | Useless parentheses. | 4 | 501 |
UselessParentheses | Useless parentheses. | 4 | 549 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 83 |
UselessParentheses | Useless parentheses. | 4 | 133 |
UselessParentheses | Useless parentheses. | 4 | 171 |
UselessParentheses | Useless parentheses. | 4 | 206 |
UselessParentheses | Useless parentheses. | 4 | 252 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 381 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'fs'. | 3 | 369 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 181 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 459–462 |
EmptyIfStmt | Avoid empty if statements | 3 | 464–469 |
UselessParentheses | Useless parentheses. | 4 | 670 |
UselessParentheses | Useless parentheses. | 4 | 748 |
UselessParentheses | Useless parentheses. | 4 | 749 |
UselessParentheses | Useless parentheses. | 4 | 750 |
UselessParentheses | Useless parentheses. | 4 | 751 |
UselessParentheses | Useless parentheses. | 4 | 752 |
UselessParentheses | Useless parentheses. | 4 | 753 |
UselessParentheses | Useless parentheses. | 4 | 812 |
UselessParentheses | Useless parentheses. | 4 | 813 |
UselessParentheses | Useless parentheses. | 4 | 813 |
UselessParentheses | Useless parentheses. | 4 | 813 |
UselessParentheses | Useless parentheses. | 4 | 909 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 921–930 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 924–929 |
UselessParentheses | Useless parentheses. | 4 | 1152–1153 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 1238–1240 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1615–1617 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 125 |
UselessParentheses | Useless parentheses. | 4 | 294 |
UselessParentheses | Useless parentheses. | 4 | 418 |
UselessParentheses | Useless parentheses. | 4 | 418 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 248–254 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 494 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'MergeBaseFailureReason': nested enums are implicitly static | 3 | 66–85 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'otp'. | 3 | 63 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 63–65 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 135–142 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 164–166 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 173–175 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 182–184 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 191–193 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 200–202 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 209–211 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 222–224 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 57–59 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 54–56 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readIncludeFile': the method is declared in an interface type | 3 | 111–112 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'sha1': the method is declared in an interface type | 3 | 157 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readFile': the method is declared in an interface type | 3 | 177–180 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readFileWithMode': the method is declared in an interface type | 3 | 204–205 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 592–594 |
UselessParentheses | Useless parentheses. | 4 | 782 |
UselessParentheses | Useless parentheses. | 4 | 783 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 162–164 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'MatchResult': nested enums are implicitly static | 3 | 62–79 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'public static final' on field 'NO_MATCH': the field is declared in an interface type | 3 | 53–64 |
Rule | Violation | Priority | Line |
---|---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 66 |
UselessParentheses | Useless parentheses. | 4 | 130 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 43–343 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Strings.getPathSeparator' due to existing static import 'org.eclipse.jgit.ignore.internal.Strings.getPathSeparator' | 4 | 131 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 138 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'PatternState': nested enums are implicitly static | 3 | 234–236 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 337–339 |
UselessParentheses | Useless parentheses. | 4 | 398 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 64–68 |
UselessParentheses | Useless parentheses. | 4 | 67 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 72–74 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 106–108 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–637 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'State': nested enums are implicitly static | 3 | 129–141 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–158 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–791 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–374 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–185 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 150–152 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–237 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–448 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'State(boolean)': enum constructors are implicitly private | 3 | 114–116 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 4 | 427 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–326 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–247 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 81–83 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 89–91 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–277 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–788 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 4 | 477 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 248–250 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 4 | 626 |
UselessParentheses | Useless parentheses. | 4 | 633 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'PackSource': nested enums are implicitly static | 3 | 100–235 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–529 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 205 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 4 | 579 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isGarbage': the method is already in a final class | 3 | 249–251 |
UselessParentheses | Useless parentheses. | 4 | 541 |
UselessParentheses | Useless parentheses. | 4 | 547 |
UselessParentheses | Useless parentheses. | 4 | 554 |
UselessParentheses | Useless parentheses. | 4 | 560 |
UselessParentheses | Useless parentheses. | 4 | 577 |
UselessParentheses | Useless parentheses. | 4 | 601 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 619 |
UselessParentheses | Useless parentheses. | 4 | 687 |
UselessParentheses | Useless parentheses. | 4 | 695 |
UselessParentheses | Useless parentheses. | 4 | 768 |
UselessParentheses | Useless parentheses. | 4 | 818 |
UselessParentheses | Useless parentheses. | 4 | 937 |
UselessParentheses | Useless parentheses. | 4 | 1047 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 183–185 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 359–361 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 379–381 |
UselessParentheses | Useless parentheses. | 4 | 762 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–182 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 377–379 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 176–178 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 240–241 |
Rule | Violation | Priority | Line |
---|---|---|---|
OverrideBothEqualsAndHashcode | Ensure you override both equals() and hashCode() | 3 | 141 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–460 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 122–124 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'clear': the method is already in a final class | 3 | 62–64 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'set': the method is already in a final class | 3 | 66–74 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'clear': the method is already in a final class | 3 | 76–80 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'get': the method is already in a final class | 3 | 82–85 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'toEWAHCompressedBitmap': the method is already in a final class | 3 | 87–112 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'block': private methods cannot be overridden | 3 | 114–116 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'mask': private methods cannot be overridden | 3 | 118–120 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofObjectType': private methods cannot be overridden | 3 | 344–346 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 112–114 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 199–201 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'InsertLooseObjectResult': nested enums are implicitly static | 3 | 63–65 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 218–221 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 291–293 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 579 |
UselessParentheses | Useless parentheses. | 4 | 579 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 422–424 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 459–461 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 577–580 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 959–961 |
UselessParentheses | Useless parentheses. | 4 | 979–981 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 984–986 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX' | 4 | 1157 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX' | 4 | 1160 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 111–113 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'maybeContains': the method is already in a final class | 3 | 76–80 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'contains': the method is already in a final class | 3 | 82–124 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'get': private methods cannot be overridden | 3 | 126–129 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'block': private methods cannot be overridden | 3 | 131–133 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'mask': private methods cannot be overridden | 3 | 135–137 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isEmpty': private methods cannot be overridden | 3 | 139–141 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'or': the method is already in a final class | 3 | 143–147 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'andNot': the method is already in a final class | 3 | 149–153 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'xor': the method is already in a final class | 3 | 155–162 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getBitmap': the method is already in a final class | 3 | 164–166 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 98–100 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 326–328 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 342–344 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 500 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 593 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 639 |
UselessParentheses | Useless parentheses. | 4 | 654 |
UselessParentheses | Useless parentheses. | 4 | 655 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 729–731 |
UselessParentheses | Useless parentheses. | 4 | 775 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 220–222 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 163 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'decompress': private methods cannot be overridden | 3 | 398–419 |
UselessParentheses | Useless parentheses. | 4 | 465 |
UselessParentheses | Useless parentheses. | 4 | 471 |
UselessParentheses | Useless parentheses. | 4 | 478 |
UselessParentheses | Useless parentheses. | 4 | 484 |
UselessParentheses | Useless parentheses. | 4 | 501 |
UselessParentheses | Useless parentheses. | 4 | 534 |
UselessParentheses | Useless parentheses. | 4 | 554 |
UselessParentheses | Useless parentheses. | 4 | 611 |
UselessParentheses | Useless parentheses. | 4 | 619 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 734–738 |
UselessParentheses | Useless parentheses. | 4 | 869 |
UselessParentheses | Useless parentheses. | 4 | 918 |
UselessParentheses | Useless parentheses. | 4 | 1033 |
UselessParentheses | Useless parentheses. | 4 | 1124 |
UselessParentheses | Useless parentheses. | 4 | 1139 |
UselessParentheses | Useless parentheses. | 4 | 1225 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 240 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 231 |
UselessParentheses | Useless parentheses. | 4 | 338 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 349–351 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 356–358 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 361–363 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'fs'. | 3 | 68 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–551 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 356 |
UselessParentheses | Useless parentheses. | 4 | 393 |
UselessParentheses | Useless parentheses. | 4 | 394 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 451–453 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 472–486 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 4 | 546 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 47–1511 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 535 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 557–558 |
UselessParentheses | Useless parentheses. | 4 | 1183 |
UselessParentheses | Useless parentheses. | 4 | 1198 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.HEAD' due to existing static import 'org.eclipse.jgit.lib.Constants.HEAD' | 4 | 1235 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_REFS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_REFS' | 4 | 1266 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'LooseRef': member interfaces are implicitly static | 3 | 1405–1409 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 217–219 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 46–274 |
UselessParentheses | Useless parentheses. | 4 | 242 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 69–73 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'bits': private methods cannot be overridden | 3 | 127–133 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 319 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 136–138 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on class 'PackInfo': members of interface types are implicitly public | 3 | 75–102 |
UnnecessaryModifier | Unnecessary modifier 'static' on class 'PackInfo': types nested within an interface type are implicitly static | 3 | 75–102 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 68 |
Rule | Violation | Priority | Line |
---|---|---|---|
ForLoopShouldBeWhileLoop | This for loop could be simplified to a while loop | 3 | 139–149 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'size': the method is already in a final class | 3 | 262–264 |
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 350 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 225 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 309–311 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'set': the method is already in a final class | 3 | 57–61 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'depth': the method is already in a final class | 3 | 64–66 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'type': the method is already in a final class | 3 | 69–71 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'size': the method is already in a final class | 3 | 74–76 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'empty': the method is already in a final class | 3 | 79–81 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'makeNext': the method is already in a final class | 3 | 83–93 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'write': the method is already in a final class | 3 | 110–114 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'write': the method is already in a final class | 3 | 118–138 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeFileHeader': the method is already in a final class | 3 | 146–153 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeObject': the method is already in a final class | 3 | 170–172 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeHeader': the method is already in a final class | 3 | 191–206 |
UselessParentheses | Useless parentheses. | 4 | 194 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'objectHeader': private methods cannot be overridden | 3 | 208–217 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofsDelta': private methods cannot be overridden | 3 | 219–226 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofsDeltaVarIntLength': private methods cannot be overridden | 3 | 228–233 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getCopyBuffer': the method is already in a final class | 3 | 240–242 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'length': the method is already in a final class | 3 | 253–255 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getDigest': the method is already in a final class | 3 | 258–260 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 45–2518 |
UselessParentheses | Useless parentheses. | 4 | 387 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1949–1951 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 2289–2290 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'have': private methods cannot be overridden | 3 | 2304–2307 |
UselessParentheses | Useless parentheses. | 4 | 2305 |
UselessParentheses | Useless parentheses. | 4 | 2306 |
UselessParentheses | Useless parentheses. | 4 | 2390 |
UselessParentheses | Useless parentheses. | 4 | 2407 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'PackingPhase': nested enums are implicitly static | 3 | 2433–2451 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 133 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE' | 4 | 234 |
UselessParentheses | Useless parentheses. | 4 | 268 |
UselessParentheses | Useless parentheses. | 4 | 269–270 |
UselessParentheses | Useless parentheses. | 4 | 270 |
UselessParentheses | Useless parentheses. | 4 | 271 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE' | 4 | 282 |
EmptyWhileStmt | Avoid empty while statements | 3 | 291–293 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 303–307 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–600 |
UselessParentheses | Useless parentheses. | 4 | 572 |
UselessParentheses | Useless parentheses. | 4 | 584 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–605 |
UselessParentheses | Useless parentheses. | 4 | 138 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–249 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–727 |
UselessParentheses | Useless parentheses. | 4 | 172 |
UselessParentheses | Useless parentheses. | 4 | 482 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 540 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 603 |
UnusedNullCheckInEquals | Invoke equals() on the object you've already ensured is not null | 3 | 710 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–859 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 707 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–342 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–417 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–227 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–287 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 43–206 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 237–238 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 372–374 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 234–236 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 446–448 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isId': the method is already in a final class | 3 | 77–87 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromString': the method is already in a final class | 3 | 101–109 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromObjectId': the method is already in a final class | 3 | 122–125 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromString': the method is already in a final class | 3 | 134–139 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromHexString': private methods cannot be overridden | 3 | 141–153 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'hexUInt32': private methods cannot be overridden | 3 | 155–166 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 3 | 247–267 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 3 | 283–303 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 3 | 319–339 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getFirstByte': the method is already in a final class | 3 | 346–348 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'name': the method is already in a final class | 3 | 376–397 |
Rule | Violation | Priority | Line |
---|---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 299 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–783 |
UselessParentheses | Useless parentheses. | 4 | 113 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on interface 'Bitmap': members of interface types are implicitly public | 3 | 77–117 |
UnnecessaryModifier | Unnecessary modifier 'public' on interface 'BitmapBuilder': members of interface types are implicitly public | 3 | 123–211 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'BranchRebaseMode(String)': enum constructors are implicitly private | 3 | 75–77 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 1443 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'SectionParser': member interfaces are implicitly static | 3 | 1476–1485 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ConfigEnum': member interfaces are implicitly static | 3 | 1515–1531 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'StringUtils.toLowerCase' due to existing static import 'org.eclipse.jgit.util.StringUtils.toLowerCase' | 4 | 281 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'AutoCRLF': nested enums are implicitly static | 3 | 63–72 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'EOL': nested enums are implicitly static | 3 | 81–90 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'EolStreamType': nested enums are implicitly static | 3 | 97–112 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'CheckStat': nested enums are implicitly static | 3 | 119–131 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'SymLinks': nested enums are implicitly static | 3 | 148–153 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'HideDotFiles': nested enums are implicitly static | 3 | 160–167 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 128–130 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 68 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 71 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 74 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 77 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 80 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 83 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 108 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 113 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 118 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 123 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 163 |
Rule | Violation | Priority | Line |
---|---|---|---|
ImportFromSamePackage | No need to import a type that lives in the same package | 3 | 45 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'GpgFormat(String)': enum constructors are implicitly private | 3 | 64–66 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'StageState': nested enums are implicitly static | 3 | 111–199 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'StageState(int)': enum constructors are implicitly private | 3 | 149–151 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getWorkingTreeIterator': the method is declared in an interface type | 3 | 352 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 474–477 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 519–523 |
UselessParentheses | Useless parentheses. | 4 | 521–522 |
UselessParentheses | Useless parentheses. | 4 | 638–639 |
UselessParentheses | Useless parentheses. | 4 | 638 |
UselessParentheses | Useless parentheses. | 4 | 638–639 |
UselessParentheses | Useless parentheses. | 4 | 767–768 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 45–1277 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH' | 4 | 571 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH' | 4 | 603 |
UselessParentheses | Useless parentheses. | 4 | 876 |
UselessParentheses | Useless parentheses. | 4 | 936 |
UselessParentheses | Useless parentheses. | 4 | 937 |
UselessParentheses | Useless parentheses. | 4 | 1015 |
UselessParentheses | Useless parentheses. | 4 | 1027 |
UselessParentheses | Useless parentheses. | 4 | 1108–1109 |
UselessParentheses | Useless parentheses. | 4 | 1110 |
UselessParentheses | Useless parentheses. | 4 | 1111 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromHexString': private methods cannot be overridden | 3 | 241–253 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'newSegment': private methods cannot be overridden | 3 | 340–342 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'computeGrowAt': private methods cannot be overridden | 3 | 344–346 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'equals': private methods cannot be overridden | 3 | 348–355 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'createArray': private methods cannot be overridden | 3 | 258–260 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Action': nested enums are implicitly static | 3 | 61–118 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Action(String, String)': enum constructors are implicitly private | 3 | 87–90 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 191–194 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Storage(boolean, boolean)': enum constructors are implicitly private | 3 | 108–111 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Result': nested enums are implicitly static | 3 | 69–172 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 782–784 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 631 |
UselessParentheses | Useless parentheses. | 4 | 643 |
UselessParentheses | Useless parentheses. | 4 | 840 |
UselessParentheses | Useless parentheses. | 4 | 841 |
UselessParentheses | Useless parentheses. | 4 | 842 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 898–899 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'java.io.File' | 4 | 46 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 299 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Key': member interfaces are implicitly static | 3 | 344–365 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 3 | 95–97 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'FetchRecurseSubmodulesMode(String)': enum constructors are implicitly private | 3 | 74–76 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–403 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 143–145 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 86 |
UselessParentheses | Useless parentheses. | 4 | 94 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 322 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 72–74 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 90 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 141 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 47–1355 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 408–410 |
UselessParentheses | Useless parentheses. | 4 | 823 |
UselessParentheses | Useless parentheses. | 4 | 823–824 |
UselessParentheses | Useless parentheses. | 4 | 823–824 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 823–849 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 839–841 |
UselessParentheses | Useless parentheses. | 4 | 887–888 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'inCore'. | 3 | 86 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException' | 4 | 51 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException' | 4 | 52 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException' | 4 | 49 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException' | 4 | 50 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 284 |
UselessParentheses | Useless parentheses. | 4 | 286 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static | 3 | 61–67 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 3 | 152–154 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–653 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'PatchType': nested enums are implicitly static | 3 | 107–116 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 248–251 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'RawParseUtils.decode' due to existing static import 'org.eclipse.jgit.util.RawParseUtils.decode' | 4 | 297 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Severity': nested enums are implicitly static | 3 | 57–63 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–392 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 195 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 198–200 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 138–141 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 157–172 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 124 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on class 'Commit': members of interface types are implicitly public | 3 | 109–145 |
UnnecessaryModifier | Unnecessary modifier 'static' on class 'Commit': types nested within an interface type are implicitly static | 3 | 109–145 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'RevWalk': members of interface types are implicitly public | 3 | 148–275 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.jgit.revwalk.RevWalk' due to existing same package import 'org.eclipse.jgit.revwalk.*' | 4 | 148 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.jgit.revwalk.ObjectWalk' due to existing same package import 'org.eclipse.jgit.revwalk.*' | 4 | 278 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'ObjectWalk': members of interface types are implicitly public | 3 | 278–393 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.lib.AnyObjectId' | 4 | 61 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 87 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 88 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 89 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 90 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 388 |
EmptyWhileStmt | Avoid empty while statements | 3 | 818–820 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 193–196 |
UselessParentheses | Useless parentheses. | 4 | 371 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 136 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 532 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 626 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 343–345 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 354–356 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 384–386 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 431–433 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 487–489 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 507–510 |
UselessParentheses | Useless parentheses. | 4 | 680 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 788–790 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 185–187 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 191–193 |
UselessParentheses | Useless parentheses. | 4 | 400–401 |
UselessParentheses | Useless parentheses. | 4 | 436 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–1971 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 380–382 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 391–393 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 407–409 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1040–1042 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1347–1349 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 264–266 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 301–303 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 3 | 114 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 315–317 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 341–343 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 425–427 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 427–429 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 429–431 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 434–436 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 439–441 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 72 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 45–561 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.OK' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.OK' | 4 | 556 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'java.util.Collection' | 4 | 50 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 95–98 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 107–110 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on class 'FtpException': types nested within an interface type are implicitly static | 3 | 64–97 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 221–223 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'MultiAck': nested enums are implicitly static | 3 | 283–285 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 175 |
UselessParentheses | Useless parentheses. | 4 | 176 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 192–194 |
EmptyIfStmt | Avoid empty if statements | 3 | 416–418 |
EmptyIfStmt | Avoid empty if statements | 3 | 419–421 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 119–137 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'HttpRedirectMode(String)': enum constructors are implicitly private | 3 | 156–158 |
UselessParentheses | Useless parentheses. | 4 | 398–399 |
UselessParentheses | Useless parentheses. | 4 | 445 |
EmptyIfStmt | Avoid empty if statements | 3 | 465–467 |
UselessParentheses | Useless parentheses. | 4 | 465 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 110–113 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 119–121 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 125–127 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 103–106 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 115–117 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 121–123 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 432–434 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 434–436 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 455–457 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 519–532 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 529–531 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 190–192 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 103–107 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'java.io.OutputStream' | 4 | 46 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.lib.ProgressMonitor' | 4 | 48 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Source': nested enums are implicitly static | 3 | 106–112 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'checkIfTooLarge': private methods cannot be overridden | 3 | 731–754 |
UselessParentheses | Useless parentheses. | 4 | 851 |
UselessParentheses | Useless parentheses. | 4 | 1026 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1184–1188 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1200–1204 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'AckNackResult': nested enums are implicitly static | 3 | 98–109 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 335–337 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 143–148 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on field 'DEFAULT': the field is declared in an interface type | 3 | 55–57 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 43–281 |
EmptyIfStmt | Avoid empty if statements | 3 | 102–104 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 267 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'StringReader': member interfaces are implicitly static | 3 | 87–98 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 265–268 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'PushRecurseSubmodulesMode(String)': enum constructors are implicitly private | 3 | 76–78 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 296–298 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static | 3 | 75–98 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Result': nested enums are implicitly static | 3 | 101–134 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 4 | 279 |
UselessParentheses | Useless parentheses. | 4 | 777 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 780–784 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 4 | 843 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 4 | 871 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 4 | 875 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 4 | 881 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 421–423 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | 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' | 4 | 167 |
UselessParentheses | Useless parentheses. | 4 | 323 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 554–555 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Status': nested enums are implicitly static | 3 | 75–127 |
UselessParentheses | Useless parentheses. | 4 | 355–356 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 154 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'TagOpt(String)': enum constructors are implicitly private | 3 | 87–89 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 162–164 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 263–265 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 89 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 118–120 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 154–156 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 214–216 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 253–255 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 182 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 353–355 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 360–362 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 47–1699 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 855–857 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1155–1157 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1176–1178 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1182–1184 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 115 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'URIishField': nested enums are implicitly static | 3 | 93–104 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 346–349 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 277–278 |
UselessParentheses | Useless parentheses. | 4 | 338 |
UselessParentheses | Useless parentheses. | 4 | 339 |
UselessParentheses | Useless parentheses. | 4 | 366 |
UselessParentheses | Useless parentheses. | 4 | 367 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 44–2414 |
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'RequestPolicy': nested enums are implicitly static | 3 | 137–167 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'IOConsumer': member interfaces are implicitly static | 3 | 236–238 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1072–1074 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1321–1323 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1330–1332 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1377–1379 |
UselessParentheses | Useless parentheses. | 4 | 1390 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1545–1547 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 120–124 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 594 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 843–845 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 302–306 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 421–423 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_OK' due to existing import 'java.net.HttpURLConnection' | 4 | 74 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_PERM' due to existing import 'java.net.HttpURLConnection' | 4 | 80 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_TEMP' due to existing import 'java.net.HttpURLConnection' | 4 | 86 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_SEE_OTHER' due to existing import 'java.net.HttpURLConnection' | 4 | 92 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_NOT_FOUND' due to existing import 'java.net.HttpURLConnection' | 4 | 105 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_UNAUTHORIZED' due to existing import 'java.net.HttpURLConnection' | 4 | 110 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_FORBIDDEN' due to existing import 'java.net.HttpURLConnection' | 4 | 115 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 45–445 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 202 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 149 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'OperationType': nested enums are implicitly static | 3 | 114–125 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 864 |
UselessParentheses | Useless parentheses. | 4 | 1302 |
UselessParentheses | Useless parentheses. | 4 | 1439 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 325–326 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 471–475 |
EmptyWhileStmt | Avoid empty while statements | 3 | 511–513 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 895–896 |
UselessParentheses | Useless parentheses. | 4 | 942 |
UselessParentheses | Useless parentheses. | 4 | 943 |
UselessParentheses | Useless parentheses. | 4 | 946 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1577–1579 |
UselessParentheses | Useless parentheses. | 4 | 1594–1595 |
UselessParentheses | Useless parentheses. | 4 | 1596–1597 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 77 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.lib.ObjectReader' | 4 | 56 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 116 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 120 |
UselessParentheses | Useless parentheses. | 4 | 127 |
UselessParentheses | Useless parentheses. | 4 | 134 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 200 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 293 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 388–390 |
EmptyIfStmt | Avoid empty if statements | 3 | 1258–1261 |
UselessParentheses | Useless parentheses. | 4 | 1332 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1875–1882 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 2214–2216 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 91 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 176–194 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 177–193 |
UselessParentheses | Useless parentheses. | 4 | 485 |
UselessParentheses | Useless parentheses. | 4 | 485 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 164–166 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 180–196 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 324–326 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 525 |
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 846 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 64 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Format': nested enums are implicitly static | 3 | 71–117 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'ParseableSimpleDateFormat(String)': enum constructors are implicitly private | 3 | 131–133 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 231–233 |
UselessParentheses | Useless parentheses. | 4 | 261–262 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'setRepository': the method is declared in an interface type | 3 | 322 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'index': private methods cannot be overridden | 3 | 169–173 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'createArray': private methods cannot be overridden | 3 | 176–178 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 91 |
UselessParentheses | Useless parentheses. | 4 | 203 |
UselessParentheses | Useless parentheses. | 4 | 206 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Status': nested enums are implicitly static | 3 | 54–69 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 45–1340 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'match': the method is already in a final class | 3 | 131–138 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseBase10': the method is already in a final class | 3 | 212–246 |
UselessParentheses | Useless parentheses. | 4 | 237 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 240–242 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseLongBase10': the method is already in a final class | 3 | 266–300 |
UselessParentheses | Useless parentheses. | 4 | 291 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 294–296 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt16': the method is already in a final class | 3 | 317–330 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt32': the method is already in a final class | 3 | 347–371 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt64': the method is already in a final class | 3 | 389–437 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt4': the method is already in a final class | 3 | 448–453 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseTimeZoneOffset': the method is already in a final class | 3 | 467–469 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseTimeZoneOffset': the method is already in a final class | 3 | 487–493 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'next': the method is already in a final class | 3 | 506–513 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'nextLF': the method is already in a final class | 3 | 526–528 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'nextLF': the method is already in a final class | 3 | 543–551 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'headerEnd': the method is already in a final class | 3 | 564–573 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'headerStart': the method is already in a final class | 3 | 588–607 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prev': the method is already in a final class | 3 | 620–628 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prevLF': the method is already in a final class | 3 | 641–643 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prevLF': the method is already in a final class | 3 | 659–668 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lineMap': the method is already in a final class | 3 | 692–700 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lineMapOrBinary': the method is already in a final class | 3 | 718–725 |
UselessParentheses | Useless parentheses. | 4 | 744 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'author': the method is already in a final class | 3 | 763–770 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'committer': the method is already in a final class | 3 | 785–794 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'tagger': the method is already in a final class | 3 | 809–822 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'encoding': the method is already in a final class | 3 | 837–847 |
UselessParentheses | Useless parentheses. | 4 | 954 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'commitMessage': the method is already in a final class | 3 | 1250–1261 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'tagMessage': the method is already in a final class | 3 | 1274–1283 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'endOfParagraph': the method is already in a final class | 3 | 1299–1309 |
UselessParentheses | Useless parentheses. | 4 | 1302 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyWhileStmt | Avoid empty while statements | 3 | 100–102 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 112 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'neq': private methods cannot be overridden | 3 | 117–119 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lc': private methods cannot be overridden | 3 | 121–123 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'isSameObject': the method is declared in an interface type | 3 | 64–66 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 122 |
UselessParentheses | Useless parentheses. | 4 | 126 |
UselessParentheses | Useless parentheses. | 4 | 127 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 405–407 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 71–73 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 149–151 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 154–156 |
Rule | Violation | Priority | Line |
---|---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 87 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'init': the method is already in a final class | 3 | 592–595 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'save': the method is already in a final class | 3 | 597–603 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'java.time.Duration' | 4 | 46 |