The following document contains the results of PMD 4.2.5.
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base3d.agent.IAgent3D' | 5 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 974 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'logger'. | 62 - 65 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.component.bus.event.impl.PausedEvent' | 8 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.component.bus.event.impl.PausingEvent' | 9 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.component.bus.event.impl.ResumedEvent' | 10 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.component.bus.event.impl.ResumingEvent' | 11 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.utils.exception.PogamutException' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.utils.exception.PogamutInterruptedException' | 78 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.PogamutProperty' | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'agentProxy'. | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.logging.Logger' | 5 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'name'. | 113 |
| These nested if statements could be combined | 118 |
| These nested if statements could be combined | 175 |
| Avoid returning from a finally block | 456 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.IWorldView' | 7 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.utils.flag.Flag' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.concurrent.ExecutionException' | 4 |
| Avoid unused imports such as 'java.util.concurrent.TimeoutException' | 7 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.agent.impl.AgentId' | 4 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.connection.IWorldConnectionAddress' | 8 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 61 - 63 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'state'. | 42 |
| Avoid unused local variables such as 'result'. | 100 |
| Avoid empty catch blocks | 132 - 133 |
| Avoid empty catch blocks | 144 - 145 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.agent.utils.runner.impl.AgentDescriptor' | 4 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Arrays' | 4 |
| Avoid empty catch blocks | 567 - 568 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.Pogamut' | 21 |
| Avoid empty catch blocks | 532 - 533 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 4 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'eventBus'. | 86 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.event.IWorldEvent' | 6 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'owner'. | 466 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.DefaultPogamutPlatform' | 19 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.PogamutPlatform' | 21 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.PogamutPlatformProxy' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.inject.ImplementedBy' | 3 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.connection.impl.socket.SocketConnection' | 6 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.Reader' | 3 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.connection.exception.ConnectionException' | 5 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.Writer' | 3 |
| Avoid unused imports such as 'com.google.inject.ImplementedBy' | 5 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.connection.exception.ConnectionException' | 7 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.utils.exception.PogamutException' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.logging.Logger' | 5 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.exception.CommunicationException' | 10 |
| These nested if statements could be combined | 129 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 7 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'reader'. | 43 |
| Avoid unused private fields such as 'eventBus'. | 53 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 6 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'eventBus'. | 56 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.concurrent.ExecutionException' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.listener.annotation.exception.MissingConstructorException' | 18 |
| Avoid empty catch blocks | 59 - 60 |
| Avoid empty catch blocks | 68 - 69 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'annotation'. | 66 - 68 |
| Avoid unused method parameters such as 'annotation'. | 74 - 76 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.concurrent.ExecutionException' | 4 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.event.IWorldEvent' | 4 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.event.IWorldEvent' | 4 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.object.IWorldObjectListener' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 5 |
| Avoid unused imports such as 'java.util.concurrent.locks.Lock' | 11 |
| Avoid unused imports such as 'java.util.concurrent.locks.ReadWriteLock' | 12 |
| Avoid unused imports such as 'java.util.concurrent.locks.ReentrantReadWriteLock' | 13 |
| Avoid unused private fields such as 'queueProcessingMutex'. | 141 |
| Avoid empty catch blocks | 238 - 239 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.component.bus.IComponentEvent' | 4 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.lang.reflect.Method' | 4 |
| Avoid unused private fields such as 'desc'. | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 4 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 112 - 113 |
| Avoid empty catch blocks | 116 - 117 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 4 |
| Avoid unused imports such as 'java.util.Set' | 6 |
| Avoid unused imports such as 'java.util.concurrent.TimeUnit' | 7 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.component.exception.ComponentKilledException' | 34 |
| Avoid unused imports such as 'cz.cuni.amis.utils.exception.PogamutInterruptedException' | 37 |
| Avoid unused imports such as 'cz.cuni.amis.utils.flag.Flag' | 38 |
| Avoid unused imports such as 'cz.cuni.amis.utils.flag.ImmutableFlag' | 39 |
| Avoid unused imports such as 'cz.cuni.amis.utils.flag.WaitForFlagChange' | 40 |
| Avoid unused imports such as 'cz.cuni.amis.utils.flag.WaitForFlagChange.IAccept' | 41 |
| Avoid empty catch blocks | 499 - 500 |
| Avoid empty catch blocks | 503 - 504 |
| Avoid empty catch blocks | 507 - 508 |
| Avoid empty catch blocks | 511 - 512 |
| Avoid empty catch blocks | 515 - 516 |
| Avoid empty catch blocks | 521 - 523 |
| Avoid empty catch blocks | 526 - 527 |
| Avoid unused method parameters such as 'states'. | 624 - 626 |
| Avoid unused private methods such as 'dependencyNotInState(IToken,ComponentState...)'. | 628 |
| Avoid unused method parameters such as 'event'. | 939 - 945 |
| Avoid unused method parameters such as 'event'. | 968 - 990 |
| Avoid unused method parameters such as 'event'. | 1004 - 1029 |
| Avoid unused method parameters such as 'event'. | 1031 - 1059 |
| These nested if statements could be combined | 1053 - 1057 |
| Avoid empty if statements | 1070 - 1072 |
| Avoid empty if statements | 1084 - 1086 |
| Avoid unused method parameters such as 'event'. | 1089 - 1105 |
| Avoid unused method parameters such as 'event'. | 1111 - 1137 |
| These nested if statements could be combined | 1131 - 1135 |
| Avoid empty if statements | 1148 - 1150 |
| Avoid empty if statements | 1162 - 1164 |
| Avoid empty if statements | 1235 - 1237 |
| Avoid empty if statements | 1249 - 1251 |
| Avoid empty if statements | 1292 - 1294 |
| Avoid empty if statements | 1306 - 1308 |
| Avoid unused method parameters such as 'manual'. | 1328 - 1340 |
| Avoid unused method parameters such as 'manual'. | 1345 - 1357 |
| Avoid unused method parameters such as 'manual'. | 1362 - 1373 |
| Avoid unused method parameters such as 'manual'. | 1379 - 1400 |
| Avoid empty catch blocks | 1382 - 1383 |
| Avoid empty catch blocks | 1386 - 1387 |
| Avoid empty catch blocks | 1390 - 1391 |
| Avoid empty catch blocks | 1394 - 1395 |
| Avoid empty catch blocks | 1398 - 1399 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1027 - 1031 |
| Avoid empty if statements | 1039 - 1041 |
| These nested if statements could be combined | 1132 - 1136 |
| Avoid empty if statements | 1144 - 1146 |
| Avoid empty catch blocks | 1274 - 1275 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'queueProcessingMutex'. | 167 |
| Avoid empty catch blocks | 264 - 265 |
| Avoid empty catch blocks | 621 - 622 |
| Avoid empty catch blocks | 633 - 634 |
| Avoid empty catch blocks | 680 - 681 |
| Avoid unused private fields such as 'ctrlMutex'. | 756 |
| Avoid unused private fields such as 'dependencies'. | 761 |
| Avoid unused private fields such as 'dependencyState'. | 766 |
| Avoid unused private fields such as 'stateCount'. | 771 |
| Avoid unused private fields such as 'componentState'. | 778 |
| Avoid unused private fields such as 'lastFatalError'. | 783 |
| Avoid unused private fields such as 'fatalErrorListener'. | 891 |
| Avoid unused private fields such as 'resetEventListener'. | 900 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.google.inject.Module' | 4 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base3d.worldview.object.event.WorldObjectAppearedEvent' | 10 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base3d.worldview.object.event.WorldObjectDisappearedEvent' | 11 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.management.AttributeChangeNotification' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.agent.jmx.AgentJMXComponents' | 11 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.agent.impl.AbstractAgent' | 6 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 94 - 95 |
| Avoid empty catch blocks | 147 - 148 |
| Avoid empty catch blocks | 198 - 199 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.management.InstanceAlreadyExistsException' | 5 |
| Avoid unused imports such as 'javax.management.MBeanRegistrationException' | 6 |
| Avoid unused imports such as 'javax.management.NotCompliantMBeanException' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.management.AttributeNotFoundException' | 3 |
| Avoid unused imports such as 'javax.management.MBeanException' | 4 |
| Avoid unused imports such as 'javax.management.ReflectionException' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.agent.jmx.AgentJMXComponents' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.logging.LogRecord' | 5 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 126 - 127 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 242 - 245 |
| Avoid empty catch blocks | 380 - 381 |
| Avoid empty catch blocks | 407 - 408 |
| Avoid empty catch blocks | 489 - 490 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.agent.IAgentId' | 12 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.nio.channels.ServerSocketChannel' | 15 |
| Avoid unused imports such as 'java.nio.channels.SocketChannel' | 16 |
| Avoid empty catch blocks | 206 - 207 |
| Avoid empty catch blocks | 446 - 447 |
| Avoid empty catch blocks | 518 - 519 |
| Avoid unused private fields such as 'writer'. | 685 |
| Avoid empty catch blocks | 799 - 800 |
| Avoid empty catch blocks | 811 - 812 |
| Avoid empty catch blocks | 851 - 852 |
| Avoid empty catch blocks | 877 - 878 |
| Avoid empty catch blocks | 883 - 884 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.EOFException' | 11 |
| Avoid unused local variables such as 'i'. | 124 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base3d.worldview.object.Rotation' | 10 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.object.IWorldObjectEvent' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.translator.event.IWorldObjectUpdateResult' | 6 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.worldview.object.ISharedWorldObject' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.IWorldView' | 7 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentScoped' | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'batches'. | 61 |
| Avoid unused private fields such as 'currentBatch'. | 66 |
| Avoid unused private fields such as 'lockRequested'. | 85 |
| Avoid unused private fields such as 'waitingForSharedBatch'. | 93 |
| Avoid unused private fields such as 'sharedWVLocks'. | 99 |
| Avoid unused private fields such as 'timeKeyIncreased'. | 101 |
| An empty statement (semicolon) not part of a loop | 471 |
| Avoid empty catch blocks | 523 - 524 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentTeamScoped' | 11 |
| Avoid unused private fields such as 'objectMutex'. | 49 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.object.event.WorldObjectDestroyedEvent' | 13 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.object.event.WorldObjectFirstEncounteredEvent' | 14 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.object.event.WorldObjectUpdatedEvent' | 15 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.worldview.object.ICompositeWorldObject' | 28 |
| An empty statement (semicolon) not part of a loop | 61 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.object.event.WorldObjectDestroyedEvent' | 18 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.object.event.WorldObjectFirstEncounteredEvent' | 19 |
| An empty statement (semicolon) not part of a loop | 234 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.worldview.ILocalWorldView' | 19 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.worldview.object.ICompositeWorldObject' | 20 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.worldview.object.ILocalWorldObject' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 6 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base3d.worldview.object.event.WorldObjectDisappearedEvent' | 17 |
| These nested if statements could be combined | 186 - 189 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentTeamScoped' | 12 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.agent.ITeamId' | 4 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.translator.event.ICompositeWorldObjectUpdatedEvent' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.translator.event.ILocalWorldObjectUpdatedEvent' | 4 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.agent.ITeamId' | 6 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.translator.event.ISharedWorldObjectUpdatedEvent' | 7 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.translator.event.IStaticWorldObjectUpdatedEvent' | 4 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.agent.ITeamId' | 4 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.translator.event.ISharedPropertyUpdateResult' | 5 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.translator.event.ISharedPropertyUpdatedEvent' | 6 |
| Avoid unused imports such as 'cz.cuni.amis.pogamut.multi.communication.translator.event.ISharedPropertyUpdateResult.Result' | 7 |
| Avoid unused imports such as 'cz.cuni.amis.utils.exception.PogamutException' | 9 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base3d.worldview.object.Rotation' | 4 |
