The following document contains the results of PMD 4.2.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.pogamut.base.utils.guice.AgentScoped' | 6 |
| 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.utils.exception.PogamutInterruptedException' | 78 |
| 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 | 461 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.IWorldView' | 7 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 601 - 604 |
| 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.utils.flag.Flag' | 5 |
| 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 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 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 '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 'java.util.Queue' | 4 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.worldview.listener.annotation.exception.MissingConstructorException' | 18 |
| Avoid empty catch blocks | 63 - 64 |
| Avoid empty catch blocks | 72 - 73 |
| 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 empty catch blocks | 112 - 113 |
| Avoid empty catch blocks | 116 - 117 |
| 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 | 486 - 487 |
| Avoid empty catch blocks | 493 - 494 |
| Avoid empty catch blocks | 498 - 499 |
| Avoid empty catch blocks | 506 - 507 |
| Avoid empty catch blocks | 510 - 511 |
| Avoid empty catch blocks | 516 - 517 |
| Avoid empty catch blocks | 520 - 521 |
| Avoid unused method parameters such as 'states'. | 617 - 619 |
| Avoid unused private methods such as 'dependencyNotInState(IToken,ComponentState...)'. | 621 |
| Avoid unused method parameters such as 'event'. | 934 - 940 |
| Avoid unused method parameters such as 'event'. | 963 - 985 |
| Avoid unused method parameters such as 'event'. | 999 - 1024 |
| Avoid unused method parameters such as 'event'. | 1026 - 1054 |
| These nested if statements could be combined | 1048 - 1052 |
| Avoid empty if statements | 1065 - 1067 |
| Avoid empty if statements | 1079 - 1081 |
| Avoid unused method parameters such as 'event'. | 1084 - 1100 |
| Avoid unused method parameters such as 'event'. | 1106 - 1132 |
| These nested if statements could be combined | 1126 - 1130 |
| Avoid empty if statements | 1143 - 1145 |
| Avoid empty if statements | 1157 - 1159 |
| Avoid empty if statements | 1231 - 1233 |
| Avoid empty if statements | 1245 - 1247 |
| Avoid empty if statements | 1288 - 1290 |
| Avoid empty if statements | 1302 - 1304 |
| Avoid unused method parameters such as 'manual'. | 1324 - 1336 |
| Avoid unused method parameters such as 'manual'. | 1341 - 1353 |
| Avoid unused method parameters such as 'manual'. | 1358 - 1369 |
| Avoid unused method parameters such as 'manual'. | 1375 - 1410 |
| Avoid empty catch blocks | 1388 - 1389 |
| Avoid empty catch blocks | 1396 - 1397 |
| Avoid empty catch blocks | 1400 - 1401 |
| Avoid empty catch blocks | 1404 - 1405 |
| Avoid empty catch blocks | 1408 - 1409 |
| 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 '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.AttributeChangeNotification' | 3 |
| 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 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 '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 unused imports such as 'cz.cuni.amis.pogamut.base3d.worldview.object.Rotation' | 10 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'cz.cuni.amis.pogamut.base3d.agent.IAgent3D' | 5 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'logger'. | 62 - 65 |
| 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 imports such as 'cz.cuni.amis.pogamut.base.utils.guice.AgentTeamScoped' | 12 |
| 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 'java.util.Queue' | 4 |
| 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.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 |
|---|---|
| These nested if statements could be combined | 193 - 197 |
| 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 |
