PMD

Bump org.apache.ant:ant from 1.10.16 to 1.10.17 #490

Triggered via push April 10, 2026 18:47

alien11689

pushed 52a7388

trunk

Status Success

Total duration 5m 6s

Artifacts –

pmd.yml

on: push

Matrix: PMD

Annotations

781 warnings and 219 notices

Error Prone UselessPureMethodCall: async/async-impl/src/main/java/org/apache/aries/async/impl/AsyncService.java#L218

Do not call pure method getConstructor if the result is not used.

Error Prone EmptyCatchBlock: async/promise-api/src/main/java/org/apache/aries/async/promise/PromiseImpl.java#L444

Avoid empty catch blocks

Error Prone EmptyCatchBlock: async/promise-api/src/main/java/org/apache/aries/async/promise/PromiseImpl.java#L465

Avoid empty catch blocks

Code Style UnnecessarySemicolon: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/container/BlueprintEvent.java#L230

Unnecessary semicolon

Design CollapsibleIfStatements: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/container/BlueprintEvent.java#L255

This if statement could be combined with its parent

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/BeanMetadata.java#L36

Unnecessary modifiers 'static final' on field 'SCOPE_SINGLETON': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/BeanMetadata.java#L43

Unnecessary modifiers 'static final' on field 'SCOPE_PROTOTYPE': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ComponentMetadata.java#L35

Unnecessary modifiers 'static final' on field 'ACTIVATION_EAGER': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ComponentMetadata.java#L42

Unnecessary modifiers 'static final' on field 'ACTIVATION_LAZY': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/NullMetadata.java#L30

Unnecessary modifiers 'static final' on field 'NULL': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ReferenceListMetadata.java#L34

Unnecessary modifiers 'static final' on field 'USE_SERVICE_OBJECT': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ReferenceListMetadata.java#L41

Unnecessary modifiers 'static final' on field 'USE_SERVICE_REFERENCE': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ServiceMetadata.java#L38

Unnecessary modifiers 'static final' on field 'AUTO_EXPORT_DISABLED': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ServiceMetadata.java#L46

Unnecessary modifiers 'static final' on field 'AUTO_EXPORT_INTERFACES': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ServiceMetadata.java#L54

Unnecessary modifiers 'static final' on field 'AUTO_EXPORT_CLASS_HIERARCHY': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ServiceMetadata.java#L62

Unnecessary modifiers 'static final' on field 'AUTO_EXPORT_ALL_CLASSES': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ServiceReferenceMetadata.java#L34

Unnecessary modifiers 'static final' on field 'AVAILABILITY_MANDATORY': the field is declared in an interface type

Code Style UnnecessaryModifier: blueprint/blueprint-api/src/main/java/org/osgi/service/blueprint/reflect/ServiceReferenceMetadata.java#L41

Unnecessary modifiers 'static final' on field 'AVAILABILITY_OPTIONAL': the field is declared in an interface type

Best Practices UnusedFormalParameter: blueprint/blueprint-authz/src/main/java/org/apache/aries/blueprint/authorization/impl/AuthorizationNsHandler.java#L38

Avoid unused method parameters such as 'cm'.

Error Prone EmptyCatchBlock: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/BaseManagedServiceFactory.java#L151

Avoid empty catch blocks

Design CollapsibleIfStatements: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmNamespaceHandler.java#L327

This if statement could be combined with its parent

Design CollapsibleIfStatements: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmNamespaceHandler.java#L368

This if statement could be combined with its parent

Design CollapsibleIfStatements: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmNamespaceHandler.java#L434

This if statement could be combined with its parent

Design CollapsibleIfStatements: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmNamespaceHandler.java#L444

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmUtils.java#L75

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmUtils.java#L99

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmUtils.java#L119

Avoid empty catch blocks

Best Practices UnusedPrivateField: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/ManagedObjectManager.java#L39

Avoid unused private fields such as 'LOGGER'.

Error Prone EmptyCatchBlock: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/ManagedObjectManager.java#L57

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/ServiceUtil.java#L32

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/AbstractServiceReferenceRecipe.java#L153

Avoid empty catch blocks

Code Style UnnecessaryModifier: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/AggregateConverter.java#L75

Unnecessary modifier 'static' on interface 'Convertible': member interfaces are implicitly static

Code Style UnnecessarySemicolon: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BeanRecipe.java#L532

Unnecessary semicolon

Error Prone AvoidBranchingStatementAsLastInLoop: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BeanRecipe.java#L945

Avoid using a branching statement as the last in a loop.

Best Practices UnusedPrivateField: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintContainerImpl.java#L118

Avoid unused private fields such as 'SECURITY_BUGFIX'.

Design CollapsibleIfStatements: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintContainerImpl.java#L356

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintContainerImpl.java#L933

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintEventDispatcher.java#L224

Avoid empty catch blocks

Design CollapsibleIfStatements: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintExtender.java#L225

This if statement could be combined with its parent

Best Practices UnusedLocalVariable: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintExtender.java#L425

Avoid unused local variables such as 'blueprintHeaderAnnotation'.

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintQuiesceParticipant.java#L62

Avoid empty catch blocks

Code Style UnnecessaryReturn: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintRepository.java#L191

Unnecessary return statement

Design CollapsibleIfStatements: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BlueprintRepository.java#L327

This if statement could be combined with its parent

Best Practices UnusedPrivateField: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/DependencyGraph.java#L37

Avoid unused private fields such as 'LOGGER'.

Best Practices UnusedPrivateField: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/ExecutorServiceWrapper.java#L36

Avoid unused private fields such as 'runningThread'.

Error Prone OverrideBothEqualsAndHashcode: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/GenericType.java#L186

Ensure you override both equals() and hashCode()

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/ReferenceListRecipe.java#L215

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/ReferenceRecipe.java#L200

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/ReferenceRecipe.java#L221

Avoid empty catch blocks

Code Style UnnecessaryModifier: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/SatisfiableRecipe.java#L38

Unnecessary modifier 'public' on interface 'SatisfactionListener': the interface is declared in an interface type

Best Practices UnusedPrivateField: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/ServiceRecipe.java#L529

Avoid unused private fields such as 'sm'.

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/di/AbstractRecipe.java#L184

Avoid empty catch blocks

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/AbstractPropertyPlaceholderExt.java#L423

Avoid unused method parameters such as 'immutable'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L239

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L257

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L272

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L286

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L300

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L321

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L365

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L383

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L401

Avoid unused method parameters such as 'context'.

Design CollapsibleIfStatements: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L513

This if statement could be combined with its parent

Best Practices UnusedPrivateMethod: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L562

Avoid unused private methods such as 'createIdRef(ParserContext, String)'.

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/namespace/NamespaceHandlerRegistryImpl.java#L323

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/namespace/NamespaceHandlerRegistryImpl.java#L425

Avoid empty catch blocks

Error Prone UnusedNullCheckInEquals: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/namespace/NamespaceHandlerRegistryImpl.java#L482

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

Design CollapsibleIfStatements: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/namespace/NamespaceHandlerRegistryImpl.java#L682

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/namespace/NamespaceHandlerRegistryImpl.java#L731

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/namespace/NamespaceHandlerRegistryImpl.java#L733

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/namespace/NamespaceHandlerRegistryImpl.java#L759

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/namespace/NamespaceHandlerRegistryImpl.java#L812

Avoid empty catch blocks

Best Practices UnusedFormalParameter: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/ReflectionUtils.java#L493

Avoid unused method parameters such as 'container'.

Error Prone AvoidMultipleUnaryOperators: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/ReflectionUtils.java#L616

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone AvoidMultipleUnaryOperators: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/ReflectionUtils.java#L651

Using multiple unary operators may be a bug, and/or is confusing.

Code Style EmptyControlStatement: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/ReflectionUtils.java#L763

Empty if statement

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/ServiceUtil.java#L30

Avoid empty catch blocks

Design CollapsibleIfStatements: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/generics/GenericsUtil.java#L210

This if statement could be combined with its parent

Best Practices UnusedPrivateMethod: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/generics/GenericsUtil.java#L672

Avoid unused private methods such as 'getClassTypes(Class<?>[])'.

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/threading/ScheduledExecutorServiceWrapper.java#L84

Avoid empty catch blocks

Error Prone AvoidMultipleUnaryOperators: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/threading/ScheduledExecutorServiceWrapper.java#L90

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone AvoidMultipleUnaryOperators: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/threading/ScheduledExecutorServiceWrapper.java#L100

Using multiple unary operators may be a bug, and/or is confusing.

Design CollapsibleIfStatements: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/threading/ScheduledExecutorServiceWrapper.java#L417

This if statement could be combined with its parent

Design CollapsibleIfStatements: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/threading/ScheduledExecutorServiceWrapper.java#L418

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/threading/impl/DiscardableCallable.java#L59

Avoid empty catch blocks

Code Style UnnecessarySemicolon: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/container/BeanRecipe.java#L641

Unnecessary semicolon

Error Prone AvoidBranchingStatementAsLastInLoop: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/container/BeanRecipe.java#L1052

Avoid using a branching statement as the last in a loop.

Error Prone OverrideBothEqualsAndHashcode: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/container/GenericType.java#L142

Ensure you override both equals() and hashCode()

Best Practices UnusedPrivateMethod: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/container/NoOsgiRecipeBuilder.java#L196

Avoid unused private methods such as 'createRecipe(RegistrationListener)'.

Best Practices UnusedPrivateMethod: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/container/NoOsgiRecipeBuilder.java#L209

Avoid unused private methods such as 'createRecipe(ReferenceListener)'.

Error Prone EmptyCatchBlock: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/container/SimpleNamespaceHandlerSet.java#L121

Avoid empty catch blocks

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/AbstractPropertyPlaceholder.java#L307

Avoid unused method parameters such as 'immutable'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L228

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L246

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L261

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L275

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L289

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L310

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L354

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L372

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L390

Avoid unused method parameters such as 'context'.

Design CollapsibleIfStatements: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L501

This if statement could be combined with its parent

Design CollapsibleIfStatements: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L583

This if statement could be combined with its parent

Best Practices UnusedPrivateMethod: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L632

Avoid unused private methods such as 'createIdRef(ParserContext, String)'.

Code Style EmptyControlStatement: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/ComponentDefinitionRegistryImpl.java#L83

Empty if statement

Code Style EmptyControlStatement: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L479

Empty if statement

Best Practices UnusedFormalParameter: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L600

Avoid unused method parameters such as 'bean'.

Design CollapsibleIfStatements: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L620

This if statement could be combined with its parent

Code Style EmptyControlStatement: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L935

Empty if statement

Code Style EmptyControlStatement: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L942

Empty if statement

Design CollapsibleIfStatements: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L1120

This if statement could be combined with its parent

Code Style EmptyControlStatement: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L1232

Empty if statement

Design CollapsibleIfStatements: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L1353

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/BeansNamespaceHandler.java#L242

Avoid empty catch blocks

Best Practices UnusedLocalVariable: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/BeansNamespaceHandler.java#L250

Avoid unused local variables such as 'importCount'.

Best Practices UnusedLocalVariable: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/BeansNamespaceHandler.java#L263

Avoid unused local variables such as 'importCount'.

Design UselessOverridingMethod: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/BlueprintBeanFactory.java#L79

Overriding method merely calls super

Best Practices UnusedFormalParameter: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/BlueprintNamespaceHandler.java#L238

Avoid unused method parameters such as 'clazz'.

Design UselessOverridingMethod: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/BlueprintNamespaceHandler.java#L311

Overriding method merely calls super

Best Practices UnusedPrivateField: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/SpringApplicationContext.java#L35

Avoid unused private fields such as 'container'.

Error Prone EmptyCatchBlock: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/SpringApplicationContext.java#L48

Avoid empty catch blocks

Best Practices UnusedPrivateMethod: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/SpringBeanProcessor.java#L90

Avoid unused private methods such as 'getProcessors(Class<T>)'.

Error Prone EmptyCatchBlock: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/SpringBeanProcessor.java#L102

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/blueprint-spring/src/main/java/org/apache/aries/blueprint/spring/SpringNamespaceHandlerResolver.java#L44

Avoid empty catch blocks

Best Practices UnusedFormalParameter: blueprint/blueprint-spring-extender/src/main/java/org/apache/aries/blueprint/spring/extender/SpringOsgiExtender.java#L109

Avoid unused method parameters such as 'bundle'.

Best Practices UnusedFormalParameter: blueprint/blueprint-spring-extender/src/main/java/org/apache/aries/blueprint/spring/extender/SpringOsgiNamespaceHandler.java#L130

Avoid unused method parameters such as 'element'.

Best Practices UnusedFormalParameter: blueprint/blueprint-spring-extender/src/main/java/org/apache/aries/blueprint/spring/extender/SpringOsgiNamespaceHandler.java#L130

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-spring-extender/src/main/java/org/apache/aries/blueprint/spring/extender/SpringOsgiNamespaceHandler.java#L134

Avoid unused method parameters such as 'element'.

Best Practices UnusedFormalParameter: blueprint/blueprint-spring-extender/src/main/java/org/apache/aries/blueprint/spring/extender/SpringOsgiNamespaceHandler.java#L134

Avoid unused method parameters such as 'context'.

Best Practices UnusedFormalParameter: blueprint/blueprint-spring-extender/src/main/java/org/apache/aries/blueprint/spring/extender/SpringOsgiNamespaceHandler.java#L139

Avoid unused method parameters such as 'element'.

Best Practices UnusedFormalParameter: blueprint/blueprint-spring-extender/src/main/java/org/apache/aries/blueprint/spring/extender/SpringOsgiNamespaceHandler.java#L139

Avoid unused method parameters such as 'context'.

Error Prone AvoidMultipleUnaryOperators: blueprint/examples/blueprint-sample/src/main/java/org/apache/aries/blueprint/sample/DestroyTest.java#L55

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone EmptyCatchBlock: blueprint/examples/blueprint-sample/src/main/java/org/apache/aries/blueprint/sample/DestroyTest.java#L58

Avoid empty catch blocks

Error Prone EmptyCatchBlock: blueprint/examples/blueprint-sample/src/main/java/org/apache/aries/blueprint/sample/DodgyListener.java#L49

Avoid empty catch blocks

Code Style UnnecessaryModifier: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/InterfaceWithDependency.java#L24

Unnecessary modifier 'public' on method 'doSomething': the method is declared in an interface type

Best Practices UnusedPrivateField: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/NSHandlerThree.java#L34

Avoid unused private fields such as 'ELT_NAME'.

Best Practices UnusedPrivateField: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/NSHandlerThree.java#L36

Avoid unused private fields such as 'ATTRIB_TWO'.

Design CollapsibleIfStatements: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/NSHandlerThree.java#L41

This if statement could be combined with its parent

Design CollapsibleIfStatements: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/NSHandlerThree.java#L42

This if statement could be combined with its parent

Best Practices UnusedPrivateField: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/NSHandlerTwo.java#L46

Avoid unused private fields such as 'ELT_NAME'.

Best Practices UnusedPrivateField: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/NSHandlerTwo.java#L48

Avoid unused private fields such as 'ATTRIB_TWO'.

Design CollapsibleIfStatements: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/NSHandlerTwo.java#L108

This if statement could be combined with its parent

Code Style UnnecessaryModifier: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/ProcessableBean.java#L29

Unnecessary modifier 'public' on enum 'Phase': the enum is declared in an interface type

Code Style UnnecessarySemicolon: blueprint/itests/blueprint-testbundlea/src/main/java/org/apache/aries/blueprint/testbundlea/ProcessableBean.java#L29

Unnecessary semicolon

Error Prone EmptyCatchBlock: blueprint/itests/blueprint-testbundleb/src/main/java/org/apache/aries/blueprint/testbundleb/OtherBean.java#L42

Avoid empty catch blocks

Best Practices UnusedPrivateField: blueprint/itests/blueprint-testbundlee/src/main/java/org/apache/aries/blueprint/testbundlee/BeanC.java#L25

Avoid unused private fields such as 'beanA'.

Best Practices UnusedPrivateField: blueprint/itests/blueprint-testbundles/src/main/java/org/apache/aries/blueprint/testbundles/BeanC.java#L25

Avoid unused private fields such as 'beanA'.

Error Prone EmptyCatchBlock: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/BundleSelector.java#L72

Avoid empty catch blocks

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L28

Unnecessary modifiers 'public static final' on field 'BUNDLE_VERSION': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L29

Unnecessary modifiers 'public static final' on field 'BUNDLE_SYMBOLICNAME': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L30

Unnecessary modifiers 'public static final' on field 'FRAGMENT_HOST': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L32

Unnecessary modifiers 'public static final' on field 'BUNDLE_TYPE': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L33

Unnecessary modifiers 'public static final' on field 'FRAGMENT_TYPE': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L38

Unnecessary modifiers 'public static final' on field 'APPLICATION_TYPE': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L39

Unnecessary modifiers 'public static final' on field 'COMPOSITE_TYPE': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L40

Unnecessary modifiers 'public static final' on field 'FEATURE_TYPE': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L45

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_MANIFESTVERSION': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L46

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_SYMBOLICNAME': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L47

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_VERSION': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L48

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_NAME': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L49

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_DESCRIPTION': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L50

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_CONTENT': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L51

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_USEBUNDLE': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L52

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_TYPE': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L54

Unnecessary modifiers 'public static final' on field 'OSGI_INF_PATH': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L55

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_MANIFEST_NAME': the field is declared in an interface type

Code Style UnnecessaryModifier: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/Constants.java#L56

Unnecessary modifiers 'public static final' on field 'SUBSYSTEM_MANIFEST_VERSION_VALUE': the field is declared in an interface type

Design CollapsibleIfStatements: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/EsaTask.java#L212

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: esa-maven-plugin/src/main/java/org/apache/aries/plugin/esa/ContentInfo.java#L101

Avoid empty catch blocks

Error Prone EmptyCatchBlock: esa-maven-plugin/src/main/java/org/apache/aries/plugin/esa/ContentInfo.java#L177

Avoid empty catch blocks

Design CollapsibleIfStatements: esa-maven-plugin/src/main/java/org/apache/aries/plugin/esa/EsaMojo.java#L568

This if statement could be combined with its parent

Code Style UnnecessaryModifier: jmx/jmx-api/src/main/java/org/osgi/jmx/framework/ServiceStateMBean.java#L196

Unnecessary modifier 'public' on method 'getObjectClass': the method is declared in an interface type

Code Style UnnecessaryModifier: jmx/jmx-api/src/main/java/org/osgi/jmx/service/provisioning/ProvisioningServiceMBean.java#L57

Unnecessary modifier 'public' on method 'addInformationFromZip': the method is declared in an interface type

Code Style UnnecessaryModifier: jmx/jmx-api/src/main/java/org/osgi/jmx/service/provisioning/ProvisioningServiceMBean.java#L71

Unnecessary modifier 'public' on method 'addInformation': the method is declared in an interface type

Code Style UnnecessaryModifier: jmx/jmx-api/src/main/java/org/osgi/jmx/service/provisioning/ProvisioningServiceMBean.java#L81

Unnecessary modifier 'public' on method 'listInformation': the method is declared in an interface type

Code Style UnnecessaryModifier: jmx/jmx-api/src/main/java/org/osgi/jmx/service/provisioning/ProvisioningServiceMBean.java#L95

Unnecessary modifier 'public' on method 'setInformation': the method is declared in an interface type

Code Style UnnecessaryModifier: jmx/jmx-blueprint-api/src/main/java/org/apache/aries/jmx/blueprint/BlueprintStateMBean.java#L190

Unnecessary modifier 'public' on method 'getLastEvent': the method is declared in an interface type

Code Style UnnecessaryModifier: jmx/jmx-blueprint-api/src/main/java/org/apache/aries/jmx/blueprint/BlueprintStateMBean.java#L198

Unnecessary modifier 'public' on method 'getLastEvents': the method is declared in an interface type

Code Style UnnecessaryModifier: jmx/jmx-blueprint-api/src/main/java/org/apache/aries/jmx/blueprint/BlueprintStateMBean.java#L207

Unnecessary modifier 'public' on method 'getBlueprintBundleIds': the method is declared in an interface type

Code Style UselessQualifiedThis: jmx/jmx-core/src/main/java/org/apache/aries/jmx/AbstractCompendiumHandler.java#L85

Useless qualified this usage in the same class.

Code Style UselessQualifiedThis: jmx/jmx-core/src/main/java/org/apache/aries/jmx/AbstractCompendiumHandler.java#L105

Useless qualified this usage in the same class.

Code Style TooManyStaticImports: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/BundleData.java#L17

Too many static imports may lead to messy code

Code Style TooManyStaticImports: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/BundleEventData.java#L17

Too many static imports may lead to messy code

Code Style TooManyStaticImports: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/PropertyData.java#L17

Too many static imports may lead to messy code

Code Style TooManyStaticImports: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/ServiceData.java#L17

Too many static imports may lead to messy code

Code Style TooManyStaticImports: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/ServiceEventData.java#L17

Too many static imports may lead to messy code

Code Style TooManyStaticImports: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/BundleState.java#L17

Too many static imports may lead to messy code

Error Prone EmptyCatchBlock: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/BundleState.java#L491

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/Framework.java#L158

Avoid empty catch blocks

Code Style UnnecessaryReturn: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/Framework.java#L285

Unnecessary return statement

Error Prone EmptyCatchBlock: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/Framework.java#L576

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/ServiceState.java#L345

Avoid empty catch blocks

Best Practices UnusedPrivateField: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/wiring/BundleWiringState.java#L42

Avoid unused private fields such as 'logger'.

Error Prone EmptyCatchBlock: jmx/jmx-core/src/main/java/org/apache/aries/jmx/useradmin/UserAdmin.java#L468

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jmx/jmx-core/src/main/java/org/apache/aries/jmx/useradmin/UserAdmin.java#L484

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jmx/jmx-core/src/main/java/org/apache/aries/jmx/useradmin/UserAdmin.java#L502

Avoid empty catch blocks

Code Style TooManyStaticImports: jmx/jmx-core/src/main/java/org/apache/aries/jmx/util/FrameworkUtils.java#L17

Too many static imports may lead to messy code

Code Style TooManyStaticImports: jmx/jmx-core/src/main/java/org/apache/aries/jmx/util/TypeUtils.java#L17

Too many static imports may lead to messy code

Code Style UnnecessaryModifier: jmx/jmx-core-whiteboard/src/main/java/org/apache/aries/jmx/core/whiteboard/Activator.java#L110

Unnecessary modifiers 'public abstract' on method 'create': the method is declared in an interface type

Error Prone EmptyCatchBlock: jmx/jmx-core-whiteboard/src/main/java/org/apache/aries/jmx/core/whiteboard/Activator.java#L240

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jmx/jmx-core-whiteboard/src/main/java/org/apache/aries/jmx/core/whiteboard/Activator.java#L296

Avoid empty catch blocks

Best Practices UnusedFormalParameter: jmx/jmx-core-whiteboard/src/main/java/org/apache/aries/jmx/core/whiteboard/Activator.java#L410

Avoid unused method parameters such as 'reference'.

Code Style UnnecessaryModifier: jndi/jndi-api/src/main/java/org/osgi/service/jndi/JNDIContextManager.java#L43

Unnecessary modifier 'public' on method 'newInitialContext': the method is declared in an interface type

Code Style UnnecessaryModifier: jndi/jndi-api/src/main/java/org/osgi/service/jndi/JNDIContextManager.java#L54

Unnecessary modifier 'public' on method 'newInitialContext': the method is declared in an interface type

Code Style UnnecessaryModifier: jndi/jndi-api/src/main/java/org/osgi/service/jndi/JNDIContextManager.java#L64

Unnecessary modifier 'public' on method 'newInitialDirContext': the method is declared in an interface type

Code Style UnnecessaryModifier: jndi/jndi-api/src/main/java/org/osgi/service/jndi/JNDIContextManager.java#L75

Unnecessary modifier 'public' on method 'newInitialDirContext': the method is declared in an interface type

Code Style UnnecessaryModifier: jndi/jndi-api/src/main/java/org/osgi/service/jndi/JNDIProviderAdmin.java#L54

Unnecessary modifier 'public' on method 'getObjectInstance': the method is declared in an interface type

Code Style UnnecessaryModifier: jndi/jndi-api/src/main/java/org/osgi/service/jndi/JNDIProviderAdmin.java#L70

Unnecessary modifier 'public' on method 'getObjectInstance': the method is declared in an interface type

Error Prone EmptyCatchBlock: jndi/jndi-core/src/main/java/org/apache/aries/jndi/ContextManagerService.java#L47

Avoid empty catch blocks

Best Practices UnusedPrivateField: jndi/jndi-core/src/main/java/org/apache/aries/jndi/ContextProvider.java#L29

Avoid unused private fields such as 'bc'.

Design CollapsibleIfStatements: jndi/jndi-core/src/main/java/org/apache/aries/jndi/ObjectFactoryHelper.java#L104

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: jndi/jndi-core/src/main/java/org/apache/aries/jndi/ObjectFactoryHelper.java#L185

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-core/src/main/java/org/apache/aries/jndi/Utils.java#L103

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-core/src/main/java/org/apache/aries/jndi/startup/Activator.java#L268

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-legacy-support/src/main/java/org/apache/aries/jndi/legacy/support/LegacyInitialContextFinder.java#L57

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-legacy-support/src/main/java/org/apache/aries/jndi/legacy/support/LegacyInitialContextFinder.java#L59

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-legacy-support/src/main/java/org/apache/aries/jndi/legacy/support/LegacyInitialContextFinder.java#L61

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-rmi/src/main/java/org/apache/aries/jndi/rmi/Activator.java#L60

Avoid empty catch blocks

Design CollapsibleIfStatements: jndi/jndi-url/src/main/java/org/apache/aries/jndi/services/ServiceHelper.java#L213

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: jndi/jndi-url/src/main/java/org/apache/aries/jndi/services/ServiceHelper.java#L263

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-url/src/main/java/org/apache/aries/jndi/services/ServiceHelper.java#L433

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-url/src/main/java/org/apache/aries/jndi/url/Activator.java#L89

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-url/src/main/java/org/apache/aries/jndi/url/BlueprintURLContext.java#L126

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-url/src/main/java/org/apache/aries/jndi/url/BlueprintURLContext.java#L131

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-url/src/main/java/org/apache/aries/jndi/url/BlueprintURLContext.java#L173

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-url/src/main/java/org/apache/aries/jndi/url/SingleServiceTracker.java#L82

Avoid empty catch blocks

Error Prone EmptyCatchBlock: jndi/jndi-url/src/main/java/org/apache/aries/jndi/url/SingleServiceTracker.java#L126

Avoid empty catch blocks

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/InvocationListener.java#L30

Unnecessary modifier 'public' on method 'preInvoke': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/InvocationListener.java#L32

Unnecessary modifier 'public' on method 'postInvoke': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/InvocationListener.java#L34

Unnecessary modifier 'public' on method 'postInvokeExceptionalReturn': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/ProxyManager.java#L45

Unnecessary modifier 'public' on method 'createDelegatingProxy': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/ProxyManager.java#L58

Unnecessary modifier 'public' on method 'createInterceptingProxy': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/ProxyManager.java#L76

Unnecessary modifier 'public' on method 'createDelegatingInterceptingProxy': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/ProxyManager.java#L85

Unnecessary modifier 'public' on method 'unwrap': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/ProxyManager.java#L92

Unnecessary modifier 'public' on method 'isProxy': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/weavinghook/ProxyWeavingController.java#L44

Unnecessary modifier 'public' on method 'shouldWeave': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/weavinghook/WeavingHelper.java#L36

Unnecessary modifier 'public' on method 'isWoven': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-api/src/main/java/org/apache/aries/proxy/weavinghook/WeavingHelper.java#L44

Unnecessary modifier 'public' on method 'isSuperClassWoven': the method is declared in an interface type

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AbstractProxyManager.java#L176

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AbstractProxyManager.java#L177

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AbstractProxyManager.java#L178

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AbstractProxyManager.java#L179

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AbstractProxyManager.java#L180

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AbstractProxyManager.java#L181

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AsmProxyManager.java#L93

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AsmProxyManager.java#L158

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AsmProxyManager.java#L165

Avoid empty catch blocks

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/JdkProxyManager.java#L39

Unnecessary modifier 'final' on method 'getInterfaces': private methods cannot be overridden

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/ProxyManagerActivator.java#L42

Avoid empty catch blocks

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/ProxyManagerActivator.java#L58

Avoid empty catch blocks

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/SingleInstanceDispatcher.java#L34

Unnecessary modifier 'final' on method 'call': the method is already in a final class

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/SystemModuleClassLoader.java#L35

Avoid empty catch blocks

Error Prone AvoidMultipleUnaryOperators: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L243

Using multiple unary operators may be a bug, and/or is confusing.

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L301

Unnecessary modifier 'final' on method 'superHasNoArgsConstructor': private methods cannot be overridden

Error Prone AvoidMultipleUnaryOperators: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L350

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone AvoidMultipleUnaryOperators: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L351

Using multiple unary operators may be a bug, and/or is confusing.

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L452

Unnecessary modifier 'final' on method 'writeFinalWovenProxyMethods': private methods cannot be overridden

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L522

Unnecessary modifier 'final' on method 'writeCreateNewProxyInstanceAndConstructor': private methods cannot be overridden

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L597

Unnecessary modifier 'final' on method 'writeStaticInitMethod': private methods cannot be overridden

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L707

Unnecessary modifier 'final' on method 'generateField': private methods cannot be overridden

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L722

Unnecessary modifier 'final' on method 'getMethodGenerator': private methods cannot be overridden

Code Style TooManyStaticImports: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L19

Too many static imports may lead to messy code

Best Practices UnusedPrivateField: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L167

Avoid unused private fields such as 'isDefaultMethod'.

Error Prone AvoidMultipleUnaryOperators: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L247

Using multiple unary operators may be a bug, and/or is confusing.

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L261

Unnecessary modifier 'final' on method 'setupLocals': private methods cannot be overridden

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L287

Unnecessary modifier 'final' on method 'beginListenerInvocation': private methods cannot be overridden

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L299

Unnecessary modifier 'final' on method 'writePreInvoke': private methods cannot be overridden

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L320

Unnecessary modifier 'final' on method 'writePostInvoke': private methods cannot be overridden

Error Prone AvoidMultipleUnaryOperators: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L333

Using multiple unary operators may be a bug, and/or is confusing.

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L348

Unnecessary modifier 'final' on method 'writeMethodCatchHandler': private methods cannot be overridden

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L474

Unnecessary modifier 'final' on method 'getAsmMethodFromClass': private methods cannot be overridden

Design CollapsibleIfStatements: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/ConstructorFinder.java#L47

This if statement could be combined with its parent

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/MethodCopyingClassAdapter.java#L96

Unnecessary modifier 'final' on method 'visitMethod': the method is already in a final class

Design CollapsibleIfStatements: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/MethodCopyingClassAdapter.java#L120

This if statement could be combined with its parent

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/MethodCopyingClassAdapter.java#L190

Unnecessary modifier 'final' on method 'visitAnnotation': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/MethodCopyingClassAdapter.java#L195

Unnecessary modifier 'final' on method 'visitAnnotationDefault': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/MethodCopyingClassAdapter.java#L200

Unnecessary modifier 'final' on method 'visitParameterAnnotation': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/MethodCopyingClassAdapter.java#L206

Unnecessary modifier 'final' on method 'visitAttribute': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/MethodCopyingClassAdapter.java#L215

Unnecessary modifier 'final' on method 'visitEnd': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/OSGiFriendlyClassWriter.java#L61

Unnecessary modifier 'final' on method 'getCommonSuperClass': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/WovenProxyAbstractMethodAdapter.java#L43

Unnecessary modifier 'final' on method 'visitCode': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/WovenProxyAbstractMethodAdapter.java#L60

Unnecessary modifier 'final' on method 'visitMaxs': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/WovenProxyAbstractMethodAdapter.java#L68

Unnecessary modifier 'final' on method 'visitEnd': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/WovenProxyConcreteMethodAdapter.java#L52

Unnecessary modifier 'final' on method 'visitCode': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/WovenProxyConcreteMethodAdapter.java#L71

Unnecessary modifier 'final' on method 'visitMaxs': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/Constants.java#L24

Unnecessary modifiers 'static final' on field 'LOG_ENTRY': the field is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/Constants.java#L25

Unnecessary modifiers 'static final' on field 'LOG_EXIT': the field is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/Constants.java#L26

Unnecessary modifiers 'static final' on field 'LOG_EXCEPTION': the field is declared in an interface type

Code Style UnnecessarySemicolon: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassGenerator.java#L178

Unnecessary semicolon

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/InterfaceCombiningClassAdapter.java#L87

Unnecessary modifier 'final' on method 'visitMethod': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/InterfaceCombiningClassAdapter.java#L113

Unnecessary modifier 'final' on method 'generateBytes': the method is already in a final class

Error Prone AvoidMultipleUnaryOperators: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/InterfaceCombiningClassAdapter.java#L114

Using multiple unary operators may be a bug, and/or is confusing.

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/InterfaceUsingWovenProxyAdapter.java#L45

Unnecessary modifier 'final' on method 'getWeavingMethodVisitor': the method is already in a final class

Error Prone EmptyCatchBlock: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/ProxyClassLoader.java#L97

Avoid empty catch blocks

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/weaving/ProxyWeavingHook.java#L88

Unnecessary modifier 'final' on method 'weave': the method is already in a final class

Design CollapsibleIfStatements: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/weaving/SyntheticSerialVerUIDAdder.java#L74

This if statement could be combined with its parent

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/weaving/WovenProxyAdapter.java#L44

Unnecessary modifier 'final' on method 'getWeavingMethodVisitor': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/weaving/WovenProxyGenerator.java#L37

Unnecessary modifier 'final' on method 'getWovenProxy': the method is already in a final class

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/weaving/WovenProxy.java#L30

Unnecessary modifier 'public' on method 'org_apache_aries_proxy_weaving_WovenProxy_isProxyInstance': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/weaving/WovenProxy.java#L35

Unnecessary modifier 'public' on method 'org_apache_aries_proxy_weaving_WovenProxy_unwrap': the method is declared in an interface type

Code Style UnnecessaryModifier: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/weaving/WovenProxy.java#L41

Unnecessary modifier 'public' on method 'org_apache_aries_proxy_weaving_WovenProxy_createNewProxyInstance': the method is declared in an interface type

Code Style TooManyStaticImports: pushstream/pushstream/src/main/java/org/apache/aries/pushstream/AbstractPushStreamImpl.java#L19

Too many static imports may lead to messy code

Code Style UnnecessaryModifier: pushstream/pushstream/src/main/java/org/apache/aries/pushstream/AbstractPushStreamImpl.java#L79

Unnecessary modifier 'static' on enum 'State': nested enums are implicitly static

Error Prone EmptyCatchBlock: pushstream/pushstream/src/main/java/org/apache/aries/pushstream/AbstractPushStreamImpl.java#L585

Avoid empty catch blocks

Error Prone EmptyCatchBlock: pushstream/pushstream/src/main/java/org/apache/aries/pushstream/AbstractPushStreamImpl.java#L602

Avoid empty catch blocks

Code Style UnnecessaryModifier: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/PushEvent.java#L45

Unnecessary modifier 'static' on enum 'EventType': nested enums are implicitly static

Code Style UnnecessaryModifier: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/PushStream.java#L280

Unnecessary modifier 'public' on method 'coalesce': the method is declared in an interface type

Code Style UnnecessaryModifier: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/PushStream.java#L292

Unnecessary modifier 'public' on method 'coalesce': the method is declared in an interface type

Code Style TooManyStaticImports: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/PushStreamProvider.java#L17

Too many static imports may lead to messy code

Error Prone EmptyCatchBlock: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/PushStreamProvider.java#L374

Avoid empty catch blocks

Error Prone EmptyCatchBlock: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/PushStreamProvider.java#L496

Avoid empty catch blocks

Code Style UnnecessaryModifier: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/PushbackPolicy.java#L47

Unnecessary modifier 'public' on method 'pushback': the method is declared in an interface type

Code Style UnnecessaryModifier: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/QueuePolicy.java#L50

Unnecessary modifier 'public' on method 'doOffer': the method is declared in an interface type

Code Style UnnecessaryModifier: quiesce/quiesce-api/src/main/java/org/apache/aries/quiesce/manager/QuiesceCallback.java#L35

Unnecessary modifier 'public' on method 'bundleQuiesced': the method is declared in an interface type

Code Style UnnecessaryModifier: quiesce/quiesce-api/src/main/java/org/apache/aries/quiesce/manager/QuiesceManager.java#L43

Unnecessary modifier 'public' on method 'quiesce': the method is declared in an interface type

Code Style UnnecessaryModifier: quiesce/quiesce-api/src/main/java/org/apache/aries/quiesce/manager/QuiesceManager.java#L50

Unnecessary modifier 'public' on method 'quiesce': the method is declared in an interface type

Code Style UnnecessaryModifier: quiesce/quiesce-api/src/main/java/org/apache/aries/quiesce/manager/QuiesceManager.java#L60

Unnecessary modifier 'public' on method 'quiesceWithFuture': the method is declared in an interface type

Code Style UnnecessaryModifier: quiesce/quiesce-api/src/main/java/org/apache/aries/quiesce/manager/QuiesceManager.java#L74

Unnecessary modifier 'public' on method 'quiesceWithFuture': the method is declared in an interface type

Code Style UnnecessaryModifier: quiesce/quiesce-api/src/main/java/org/apache/aries/quiesce/participant/QuiesceParticipant.java#L41

Unnecessary modifier 'public' on method 'quiesce': the method is declared in an interface type

Error Prone AvoidMultipleUnaryOperators: quiesce/quiesce-manager/src/main/java/org/apache/aries/quiesce/manager/impl/QuiesceManagerImpl.java#L100

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone AvoidMultipleUnaryOperators: quiesce/quiesce-manager/src/main/java/org/apache/aries/quiesce/manager/impl/QuiesceManagerImpl.java#L137

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone AvoidMultipleUnaryOperators: quiesce/quiesce-manager/src/main/java/org/apache/aries/quiesce/manager/impl/QuiesceManagerImpl.java#L152

Using multiple unary operators may be a bug, and/or is confusing.

Design CollapsibleIfStatements: quiesce/quiesce-manager/src/main/java/org/apache/aries/quiesce/manager/impl/QuiesceManagerImpl.java#L316

This if statement could be combined with its parent

Error Prone AvoidMultipleUnaryOperators: quiesce/quiesce-manager/src/main/java/org/apache/aries/quiesce/manager/impl/QuiesceManagerImpl.java#L343

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone AvoidMultipleUnaryOperators: quiesce/quiesce-manager/src/main/java/org/apache/aries/quiesce/manager/impl/QuiesceManagerImpl.java#L353

Using multiple unary operators may be a bug, and/or is confusing.

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeDBManager.java#L38

Unnecessary modifier 'public' on method 'checkDBProductName': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeDBManager.java#L48

Unnecessary modifier 'public' on method 'recreateDBTables': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeDBManager.java#L58

Unnecessary modifier 'public' on method 'resetTrade': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L55

Unnecessary modifier 'public' on method 'getMarketSummary': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L72

Unnecessary modifier 'public' on method 'buy': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L83

Unnecessary modifier 'public' on method 'sell': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L98

Unnecessary modifier 'public' on method 'queueOrder': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L113

Unnecessary modifier 'public' on method 'completeOrder': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L124

Unnecessary modifier 'public' on method 'cancelOrder': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L134

Unnecessary modifier 'public' on method 'orderCompleted': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L143

Unnecessary modifier 'public' on method 'getOrders': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L151

Unnecessary modifier 'public' on method 'getClosedOrders': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L162

Unnecessary modifier 'public' on method 'createQuote': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L170

Unnecessary modifier 'public' on method 'getQuote': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L177

Unnecessary modifier 'public' on method 'getAllQuotes': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L186

Unnecessary modifier 'public' on method 'updateQuotePriceVolume': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L196

Unnecessary modifier 'public' on method 'getHoldings': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L204

Unnecessary modifier 'public' on method 'getHolding': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L212

Unnecessary modifier 'public' on method 'getAccountData': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L221

Unnecessary modifier 'public' on method 'getAccountProfileData': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L233

Unnecessary modifier 'public' on method 'updateAccountProfile': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L243

Unnecessary modifier 'public' on method 'login': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L252

Unnecessary modifier 'public' on method 'logout': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L268

Unnecessary modifier 'public' on method 'register': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServices.java#L282

Unnecessary modifier 'public' on method 'getMode': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServicesManager.java#L35

Unnecessary modifier 'public' on method 'getCurrentModes': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServicesManager.java#L40

Unnecessary modifier 'public' on method 'getTradeServices': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/TradeServicesManager.java#L55

Unnecessary modifier 'public' on method 'getMarketSummary': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L25

Unnecessary modifier 'public' on method 'toString': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L27

Unnecessary modifier 'public' on method 'toHTML': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L29

Unnecessary modifier 'public' on method 'getAccountID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L31

Unnecessary modifier 'public' on method 'setAccountID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L33

Unnecessary modifier 'public' on method 'getLoginCount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L35

Unnecessary modifier 'public' on method 'setLoginCount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L37

Unnecessary modifier 'public' on method 'getLogoutCount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L39

Unnecessary modifier 'public' on method 'setLogoutCount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L41

Unnecessary modifier 'public' on method 'getLastLogin': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L43

Unnecessary modifier 'public' on method 'setLastLogin': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L45

Unnecessary modifier 'public' on method 'getCreationDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L47

Unnecessary modifier 'public' on method 'setCreationDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L49

Unnecessary modifier 'public' on method 'getBalance': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L51

Unnecessary modifier 'public' on method 'setBalance': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L53

Unnecessary modifier 'public' on method 'getOpenBalance': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L55

Unnecessary modifier 'public' on method 'setOpenBalance': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L57

Unnecessary modifier 'public' on method 'getProfileID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L59

Unnecessary modifier 'public' on method 'setProfileID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L61

Unnecessary modifier 'public' on method 'getOrders': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L63

Unnecessary modifier 'public' on method 'getHoldings': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L65

Unnecessary modifier 'public' on method 'getProfile': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L67

Unnecessary modifier 'public' on method 'setProfile': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L69

Unnecessary modifier 'public' on method 'login': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountDataBean.java#L71

Unnecessary modifier 'public' on method 'logout': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L21

Unnecessary modifier 'public' on method 'toString': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L23

Unnecessary modifier 'public' on method 'toHTML': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L25

Unnecessary modifier 'public' on method 'getUserID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L27

Unnecessary modifier 'public' on method 'setUserID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L29

Unnecessary modifier 'public' on method 'getPassword': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L31

Unnecessary modifier 'public' on method 'setPassword': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L33

Unnecessary modifier 'public' on method 'getFullName': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L35

Unnecessary modifier 'public' on method 'setFullName': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L37

Unnecessary modifier 'public' on method 'getAddress': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L39

Unnecessary modifier 'public' on method 'setAddress': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L41

Unnecessary modifier 'public' on method 'getEmail': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L43

Unnecessary modifier 'public' on method 'setEmail': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L45

Unnecessary modifier 'public' on method 'getCreditCard': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L47

Unnecessary modifier 'public' on method 'setCreditCard': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L49

Unnecessary modifier 'public' on method 'getAccount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/AccountProfileDataBean.java#L51

Unnecessary modifier 'public' on method 'setAccount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L24

Unnecessary modifier 'public' on method 'toString': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L26

Unnecessary modifier 'public' on method 'toHTML': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L28

Unnecessary modifier 'public' on method 'getHoldingID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L30

Unnecessary modifier 'public' on method 'setHoldingID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L32

Unnecessary modifier 'public' on method 'getQuantity': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L34

Unnecessary modifier 'public' on method 'setQuantity': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L36

Unnecessary modifier 'public' on method 'getPurchasePrice': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L38

Unnecessary modifier 'public' on method 'setPurchasePrice': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L40

Unnecessary modifier 'public' on method 'getPurchaseDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L42

Unnecessary modifier 'public' on method 'setPurchaseDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L44

Unnecessary modifier 'public' on method 'getQuoteID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L46

Unnecessary modifier 'public' on method 'setQuoteID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L48

Unnecessary modifier 'public' on method 'getAccount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L50

Unnecessary modifier 'public' on method 'setAccount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L52

Unnecessary modifier 'public' on method 'getQuote': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/HoldingDataBean.java#L54

Unnecessary modifier 'public' on method 'setQuote': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L24

Unnecessary modifier 'public' on method 'toString': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L26

Unnecessary modifier 'public' on method 'toHTML': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L28

Unnecessary modifier 'public' on method 'getOrderID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L30

Unnecessary modifier 'public' on method 'setOrderID': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L32

Unnecessary modifier 'public' on method 'getOrderType': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L34

Unnecessary modifier 'public' on method 'setOrderType': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L36

Unnecessary modifier 'public' on method 'getOrderStatus': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L38

Unnecessary modifier 'public' on method 'setOrderStatus': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L40

Unnecessary modifier 'public' on method 'getOpenDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L42

Unnecessary modifier 'public' on method 'setOpenDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L44

Unnecessary modifier 'public' on method 'getCompletionDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L46

Unnecessary modifier 'public' on method 'setCompletionDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L48

Unnecessary modifier 'public' on method 'getQuantity': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L50

Unnecessary modifier 'public' on method 'setQuantity': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L52

Unnecessary modifier 'public' on method 'getPrice': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L54

Unnecessary modifier 'public' on method 'setPrice': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L56

Unnecessary modifier 'public' on method 'getOrderFee': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L58

Unnecessary modifier 'public' on method 'setOrderFee': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L60

Unnecessary modifier 'public' on method 'getSymbol': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L62

Unnecessary modifier 'public' on method 'setSymbol': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L64

Unnecessary modifier 'public' on method 'getAccount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L66

Unnecessary modifier 'public' on method 'setAccount': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L68

Unnecessary modifier 'public' on method 'getQuote': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L70

Unnecessary modifier 'public' on method 'setQuote': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L72

Unnecessary modifier 'public' on method 'getHolding': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L74

Unnecessary modifier 'public' on method 'setHolding': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L76

Unnecessary modifier 'public' on method 'isBuy': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L78

Unnecessary modifier 'public' on method 'isSell': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L80

Unnecessary modifier 'public' on method 'isOpen': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L82

Unnecessary modifier 'public' on method 'isCompleted': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L84

Unnecessary modifier 'public' on method 'isCancelled': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/OrderDataBean.java#L86

Unnecessary modifier 'public' on method 'cancel': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L23

Unnecessary modifier 'public' on method 'toString': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L25

Unnecessary modifier 'public' on method 'toHTML': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L27

Unnecessary modifier 'public' on method 'getSymbol': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L29

Unnecessary modifier 'public' on method 'setSymbol': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L31

Unnecessary modifier 'public' on method 'getCompanyName': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L33

Unnecessary modifier 'public' on method 'setCompanyName': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L35

Unnecessary modifier 'public' on method 'getPrice': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L37

Unnecessary modifier 'public' on method 'setPrice': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L39

Unnecessary modifier 'public' on method 'getOpen': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L41

Unnecessary modifier 'public' on method 'setOpen': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L43

Unnecessary modifier 'public' on method 'getLow': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L45

Unnecessary modifier 'public' on method 'setLow': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L47

Unnecessary modifier 'public' on method 'getHigh': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L49

Unnecessary modifier 'public' on method 'setHigh': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L51

Unnecessary modifier 'public' on method 'getChange': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L53

Unnecessary modifier 'public' on method 'setChange': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L55

Unnecessary modifier 'public' on method 'getVolume': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-api/src/main/java/org/apache/aries/samples/ariestrader/api/persistence/QuoteDataBean.java#L57

Unnecessary modifier 'public' on method 'setVolume': the method is declared in an interface type

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-beans/src/main/java/org/apache/aries/samples/ariestrader/beans/AccountDataBeanImpl.java#L107

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-beans/src/main/java/org/apache/aries/samples/ariestrader/beans/HoldingDataBeanImpl.java#L84

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-beans/src/main/java/org/apache/aries/samples/ariestrader/beans/OrderDataBeanImpl.java#L115

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L56

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices CheckResultSet: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L237

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L247

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L256

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L277

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L289

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L302

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L315

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L335

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Code Style UnnecessaryReturn: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeDBManagerImpl.java#L453

Unnecessary return statement

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-core/src/main/java/org/apache/aries/samples/ariestrader/core/TradeServicesManagerImpl.java#L48

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/AccountDataBeanImpl.java#L150

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/HoldingDataBeanImpl.java#L101

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/OrderDataBeanImpl.java#L147

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L74

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L82

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L216

Do not use `new Double(...)`, prefer `Double.valueOf(...)`

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L230

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L301

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Best Practices UnusedLocalVariable: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L394

Avoid unused local variables such as 'orderFee'.

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L403

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L504

Avoid unused method parameters such as 'orderID'.

Best Practices UnusedPrivateMethod: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L866

Avoid unused private methods such as 'getAccountDataForUpdate(Connection, String)'.

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L883

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L1050

Avoid unused method parameters such as 'symbol'.

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L1097

Do not use `new Double(...)`, prefer `Double.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L1127

Do not use `new Double(...)`, prefer `Double.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L1333

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L1356

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L1375

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Design CollapsibleIfStatements: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L1434

This if statement could be combined with its parent

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L194

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L884

Avoid unused method parameters such as 'quote'.

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L885

Avoid unused method parameters such as 'oldPrice'.

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L885

Avoid unused method parameters such as 'changeFactor'.

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L885

Avoid unused method parameters such as 'sharesTraded'.

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L966

Do not use `new Double(...)`, prefer `Double.valueOf(...)`

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L176

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L677

Avoid unused method parameters such as 'quote'.

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L678

Avoid unused method parameters such as 'oldPrice'.

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L678

Avoid unused method parameters such as 'changeFactor'.

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L678

Avoid unused method parameters such as 'sharesTraded'.

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L743

Do not use `new Double(...)`, prefer `Double.valueOf(...)`

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/FinancialUtils.java#L26

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/FinancialUtils.java#L27

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/FinancialUtils.java#L28

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/KeyBlock.java#L78

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/KeyBlock.java#L92

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Code Style UnnecessaryModifier: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L38

Unnecessary modifier 'static' on enum 'ModeType': nested enums are implicitly static

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L133

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L136

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Error Prone UselessPureMethodCall: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L137

Do not call pure method setScale if the result is not used.

Error Prone UselessPureMethodCall: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L151

Do not call pure method setScale if the result is not used.

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L152

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Error Prone UselessPureMethodCall: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L153

Do not call pure method setScale if the result is not used.

Error Prone EmptyCatchBlock: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L236

Avoid empty catch blocks

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L368

Do not use `new Float(...)`, prefer `Float.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L371

Do not use `new Float(...)`, prefer `Float.valueOf(...)`

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L374

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L396

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L398

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L407

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L415

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L447

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices UnusedLocalVariable: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/OrdersAlertFilter.java#L96

Avoid unused local variables such as 'sc'.

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeAppServlet.java#L165

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeBuildDB.java#L172

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Error Prone AvoidDecimalLiteralsInBigDecimalConstructor: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeBuildDB.java#L210

Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal

Design UselessOverridingMethod: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeConfigServlet.java#L45

Overriding method merely calls super

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L219

Do not use `new Double(...)`, prefer `Double.valueOf(...)`

Best Practices UnusedLocalVariable: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L541

Avoid unused local variables such as 'results'.

Best Practices UnusedFormalParameter: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L691

Avoid unused method parameters such as 'userID'.

Best Practices UnusedLocalVariable: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/prims/PingServlet2JNDI.java#L72

Avoid unused local variables such as 'dataSource'.

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/prims/PingSession1.java#L79

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/prims/PingSession2.java#L79

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/prims/PingSession2.java#L83

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogAuthorManager.java#L36

Unnecessary modifier 'public' on method 'createAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogAuthorManager.java#L42

Unnecessary modifier 'public' on method 'getAllAuthors': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogAuthorManager.java#L49

Unnecessary modifier 'public' on method 'getAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogAuthorManager.java#L55

Unnecessary modifier 'public' on method 'removeAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogAuthorManager.java#L66

Unnecessary modifier 'public' on method 'updateAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogCommentManager.java#L30

Unnecessary modifier 'public' on method 'createComment': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogCommentManager.java#L37

Unnecessary modifier 'public' on method 'getCommentsByAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogCommentManager.java#L44

Unnecessary modifier 'public' on method 'getCommentsForPost': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogCommentManager.java#L50

Unnecessary modifier 'public' on method 'deleteComment': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogCommentManager.java#L56

Unnecessary modifier 'public' on method 'isCommentingAvailable': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntry.java#L53

Unnecessary modifier 'public' on method 'getPublishDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntry.java#L59

Unnecessary modifier 'public' on method 'getId': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L37

Unnecessary modifier 'public' on method 'createBlogPost': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L44

Unnecessary modifier 'public' on method 'findBlogEntryByTitle': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L50

Unnecessary modifier 'public' on method 'getAllBlogEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L57

Unnecessary modifier 'public' on method 'getBlogsForAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L66

Unnecessary modifier 'public' on method 'getBlogEntriesModifiedBetween': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L75

Unnecessary modifier 'public' on method 'getBlogEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L81

Unnecessary modifier 'public' on method 'getNoOfPosts': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L90

Unnecessary modifier 'public' on method 'removeBlogEntry': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L102

Unnecessary modifier 'public' on method 'updateBlogEntry': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BlogEntryManager.java#L110

Unnecessary modifier 'public' on method 'getBlogPost': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BloggingService.java#L63

Unnecessary modifier 'public' on method 'getNoOfEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BloggingService.java#L71

Unnecessary modifier 'public' on method 'getBlogEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/BloggingService.java#L77

Unnecessary modifier 'public' on method 'getAllBlogEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Author.java#L28

Unnecessary modifier 'public' on method 'getEmail': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Author.java#L31

Unnecessary modifier 'public' on method 'getName': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Author.java#L34

Unnecessary modifier 'public' on method 'getDisplayName': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Author.java#L37

Unnecessary modifier 'public' on method 'getBio': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Author.java#L40

Unnecessary modifier 'public' on method 'getDob': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Author.java#L43

Unnecessary modifier 'public' on method 'getEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L39

Unnecessary modifier 'public' on method 'getAllBlogEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L45

Unnecessary modifier 'public' on method 'getNoOfBlogEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L52

Unnecessary modifier 'public' on method 'getBlogEntries': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L61

Unnecessary modifier 'public' on method 'getBlogsForAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L69

Unnecessary modifier 'public' on method 'findBlogEntryByTitle': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L77

Unnecessary modifier 'public' on method 'getBlogEntriesModifiedBetween': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L85

Unnecessary modifier 'public' on method 'getBlogEntryById': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L92

Unnecessary modifier 'public' on method 'getAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L98

Unnecessary modifier 'public' on method 'getAllAuthors': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L114

Unnecessary modifier 'public' on method 'createAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L128

Unnecessary modifier 'public' on method 'createBlogPost': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L134

Unnecessary modifier 'public' on method 'updateAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L145

Unnecessary modifier 'public' on method 'updateBlogEntry': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L152

Unnecessary modifier 'public' on method 'removeAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/BlogPersistenceService.java#L160

Unnecessary modifier 'public' on method 'removeBlogEntry': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Entry.java#L29

Unnecessary modifier 'public' on method 'getAuthor': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Entry.java#L32

Unnecessary modifier 'public' on method 'getPublishDate': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Entry.java#L35

Unnecessary modifier 'public' on method 'getTitle': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Entry.java#L38

Unnecessary modifier 'public' on method 'getTags': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Entry.java#L41

Unnecessary modifier 'public' on method 'getBlogText': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Entry.java#L44

Unnecessary modifier 'public' on method 'getId': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blog/blog-api/src/main/java/org/apache/aries/samples/blog/api/persistence/Entry.java#L50

Unnecessary modifier 'public' on method 'getUpdatedDate': the method is declared in an interface type

Best Practices UnusedFormalParameter: samples/blog/blog-biz/src/main/java/org/apache/aries/samples/blog/biz/BlogAuthorImpl.java#L77

Avoid unused method parameters such as 'list'.

Code Style UnnecessarySemicolon: samples/blog/blog-biz/src/main/java/org/apache/aries/samples/blog/biz/BlogAuthorImpl.java#L79

Unnecessary semicolon

Error Prone EmptyCatchBlock: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L85

Avoid empty catch blocks

Error Prone EmptyCatchBlock: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L91

Avoid empty catch blocks

Error Prone EmptyCatchBlock: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L117

Avoid empty catch blocks

Error Prone EmptyCatchBlock: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L123

Avoid empty catch blocks

Best Practices CheckResultSet: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L197

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L310

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Best Practices CheckResultSet: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L487

Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet.

Error Prone AvoidMultipleUnaryOperators: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L532

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone AvoidMultipleUnaryOperators: samples/blog/blog-web/src/main/java/org/apache/aries/samples/blog/web/EditAuthorForm.java#L57

Using multiple unary operators may be a bug, and/or is confusing.

Error Prone AvoidMultipleUnaryOperators: samples/blog/blog-web/src/main/java/org/apache/aries/samples/blog/web/EditAuthorForm.java#L91

Using multiple unary operators may be a bug, and/or is confusing.

Code Style EmptyControlStatement: samples/blog/blog-web/src/main/java/org/apache/aries/samples/blog/web/ViewAuthor.java#L45

Empty if statement

Error Prone AvoidMultipleUnaryOperators: samples/blog/blog-web/src/main/java/org/apache/aries/samples/blog/web/util/HTMLOutput.java#L154

Using multiple unary operators may be a bug, and/or is confusing.

Code Style UnnecessaryModifier: samples/blueprint/idverifier/idverifier-api/src/main/java/org/apache/aries/samples/blueprint/idverifier/api/CreditRecordOperation.java#L28

Unnecessary modifier 'public' on method 'query': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blueprint/idverifier/idverifier-api/src/main/java/org/apache/aries/samples/blueprint/idverifier/api/CreditRecordOperation.java#L29

Unnecessary modifier 'public' on method 'add': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blueprint/idverifier/idverifier-api/src/main/java/org/apache/aries/samples/blueprint/idverifier/api/CreditRecordOperation.java#L30

Unnecessary modifier 'public' on method 'remove': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blueprint/idverifier/idverifier-api/src/main/java/org/apache/aries/samples/blueprint/idverifier/api/PersonIDVerifier.java#L26

Unnecessary modifier 'public' on method 'setId': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blueprint/idverifier/idverifier-api/src/main/java/org/apache/aries/samples/blueprint/idverifier/api/PersonIDVerifier.java#L27

Unnecessary modifier 'public' on method 'verify': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blueprint/idverifier/idverifier-api/src/main/java/org/apache/aries/samples/blueprint/idverifier/api/PersonIDVerifier.java#L28

Unnecessary modifier 'public' on method 'getArea': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blueprint/idverifier/idverifier-api/src/main/java/org/apache/aries/samples/blueprint/idverifier/api/PersonIDVerifier.java#L29

Unnecessary modifier 'public' on method 'getBirthday': the method is declared in an interface type

Code Style UnnecessaryModifier: samples/blueprint/idverifier/idverifier-api/src/main/java/org/apache/aries/samples/blueprint/idverifier/api/PersonIDVerifier.java#L30

Unnecessary modifier 'public' on method 'getGender': the method is declared in an interface type

Error Prone UselessPureMethodCall: samples/blueprint/idverifier/idverifier-client/src/main/java/org/apache/aries/samples/blueprint/idverifier/client/PersonBankBean.java#L139

Do not call pure method toString if the result is not used.

Error Prone UselessPureMethodCall: samples/blueprint/idverifier/idverifier-client/src/main/java/org/apache/aries/samples/blueprint/idverifier/client/PersonBankBean.java#L143

Do not call pure method toString if the result is not used.

Design CollapsibleIfStatements: samples/blueprint/idverifier/idverifier-client/src/main/java/org/apache/aries/samples/blueprint/idverifier/client/PersonCreditRecords.java#L42

This if statement could be combined with its parent

Design CollapsibleIfStatements: samples/blueprint/idverifier/idverifier-client/src/main/java/org/apache/aries/samples/blueprint/idverifier/client/PersonCreditRecords.java#L53

This if statement could be combined with its parent

Best Practices PrimitiveWrapperInstantiation: samples/blueprint/idverifier/idverifier-server/src/main/java/org/apache/aries/samples/blueprint/idverifier/server/PersonIDVerifierComplexImpl.java#L101

Do not use `new Character(...)`, prefer `Character.valueOf(...)`

Design CollapsibleIfStatements: spi-fly/spi-fly-core/src/main/java/org/apache/aries/spifly/BaseActivator.java#L386

This if statement could be combined with its parent

Design CollapsibleIfStatements: spi-fly/spi-fly-core/src/main/java/org/apache/aries/spifly/ConsumerHeaderProcessor.java#L250

This if statement could be combined with its parent

Error Prone AvoidBranchingStatementAsLastInLoop: spi-fly/spi-fly-core/src/main/java/org/apache/aries/spifly/ProviderBundleTrackerCustomizer.java#L411

Avoid using a branching statement as the last in a loop.

Error Prone EmptyCatchBlock: spi-fly/spi-fly-core/src/main/java/org/apache/aries/spifly/ProviderBundleTrackerCustomizer.java#L459

Avoid empty catch blocks

Error Prone EmptyCatchBlock: spi-fly/spi-fly-core/src/main/java/org/apache/aries/spifly/Util.java#L253

Avoid empty catch blocks

Error Prone EmptyCatchBlock: spi-fly/spi-fly-core/src/main/java/org/apache/aries/spifly/Util.java#L365

Avoid empty catch blocks

Code Style UnnecessaryModifier: spi-fly/spi-fly-dynamic-bundle/src/main/java/org/apache/aries/spifly/dynamic/OSGiFriendlyClassWriter.java#L65

Unnecessary modifier 'final' on method 'getCommonSuperClass': the method is already in a final class

Error Prone EmptyCatchBlock: spi-fly/spi-fly-examples/spi-fly-example-provider-consumer-bundle/src/main/java/org/apache/aries/spifly/pc/bundle/Activator.java#L38

Avoid empty catch blocks

Design CollapsibleIfStatements: spi-fly/spi-fly-static-tool/src/main/java/org/apache/aries/spifly/statictool/Main.java#L351

This if statement could be combined with its parent

Best Practices UnusedPrivateField: spi-fly/spi-fly-weaver/src/main/java/org/apache/aries/spifly/weaver/TCCLSetterVisitor.java#L158

Avoid unused private fields such as 'lastOpcode'.

Best Practices UnusedPrivateField: spi-fly/spi-fly-weaver/src/main/java/org/apache/aries/spifly/weaver/TCCLSetterVisitor.java#L159

Avoid unused private fields such as 'lastVar'.

Design SimplifiedTernary: spi-fly/spi-fly-weaver/src/main/java/org/apache/aries/spifly/weaver/TCCLSetterVisitor.java#L294

This conditional expression can be simplified with || or &&

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/apache/aries/subsystem/ContentHandler.java#L31

Unnecessary modifiers 'static final' on field 'CONTENT_TYPE_PROPERTY': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/apache/aries/subsystem/ContentHandler.java#L32

Unnecessary modifiers 'static final' on field 'EMBEDDED_RESOURCE_ATTRIBUTE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L173

Unnecessary modifier 'public' on enum 'State': the enum is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L173

Unnecessary modifier 'static' on enum 'State': nested enums are implicitly static

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L291

Unnecessary modifier 'public' on method 'getBundleContext': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L304

Unnecessary modifier 'public' on method 'getChildren': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L330

Unnecessary modifier 'public' on method 'getSubsystemHeaders': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L349

Unnecessary modifier 'public' on method 'getLocation': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L364

Unnecessary modifier 'public' on method 'getParents': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L377

Unnecessary modifier 'public' on method 'getConstituents': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L399

Unnecessary modifier 'public' on method 'getDeploymentHeaders': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L410

Unnecessary modifier 'public' on method 'getState': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L425

Unnecessary modifier 'public' on method 'getSubsystemId': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L451

Unnecessary modifier 'public' on method 'getSymbolicName': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L463

Unnecessary modifier 'public' on method 'getType': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L489

Unnecessary modifier 'public' on method 'getVersion': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L510

Unnecessary modifier 'public' on method 'install': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L610

Unnecessary modifier 'public' on method 'install': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L643

Unnecessary modifier 'public' on method 'install': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L759

Unnecessary modifier 'public' on method 'start': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L862

Unnecessary modifier 'public' on method 'stop': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/Subsystem.java#L958

Unnecessary modifier 'public' on method 'uninstall': the method is declared in an interface type

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/SubsystemPermission.java#L590

Do not use `new Long(...)`, prefer `Long.valueOf(...)`

Design CollapsibleIfStatements: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/SubsystemPermission.java#L668

This if statement could be combined with its parent

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/AbstractClauseBasedHeader.java#L91

Unnecessary modifier 'public' on method 'newInstance': the method is declared in an interface type

Best Practices UnusedFormalParameter: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/AriesSubsystemParentsHeader.java#L81

Avoid unused method parameters such as 'referenceCount'.

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Attribute.java#L17

Unnecessary modifier 'public' on method 'appendToFilter': the method is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L76

Unnecessary modifiers 'public static final' on field 'DIGIT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L77

Unnecessary modifiers 'public static final' on field 'ALPHA': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L78

Unnecessary modifiers 'public static final' on field 'ALPHANUM': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L79

Unnecessary modifiers 'public static final' on field 'TOKEN': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L80

Unnecessary modifiers 'public static final' on field 'EXTENDED': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L81

Unnecessary modifiers 'public static final' on field 'QUOTED_STRING': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L82

Unnecessary modifiers 'public static final' on field 'ARGUMENT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L83

Unnecessary modifiers 'public static final' on field 'DIRECTIVE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L84

Unnecessary modifiers 'public static final' on field 'ATTRIBUTE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L85

Unnecessary modifiers 'public static final' on field 'PARAMETER': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L86

Unnecessary modifiers 'public static final' on field 'PATH_ELEMENT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L87

Unnecessary modifiers 'public static final' on field 'PATH_ELEMENT_NT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L88

Unnecessary modifiers 'public static final' on field 'PATH_SEP': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L89

Unnecessary modifiers 'public static final' on field 'PATH_UNQUOTED': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L90

Unnecessary modifiers 'public static final' on field 'PATH_UNQUOTED_NT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L91

Unnecessary modifiers 'public static final' on field 'PATH': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L92

Unnecessary modifiers 'public static final' on field 'CLAUSE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L93

Unnecessary modifiers 'public static final' on field 'HEADERCHAR': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L94

Unnecessary modifiers 'public static final' on field 'NAME': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L95

Unnecessary modifiers 'public static final' on field 'HEADER': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L108

Unnecessary modifiers 'public static final' on field 'JLETTER': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L109

Unnecessary modifiers 'public static final' on field 'JLETTERORDIGIT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L110

Unnecessary modifiers 'public static final' on field 'IDENTIFIER': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L111

Unnecessary modifiers 'public static final' on field 'UNIQUENAME': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L112

Unnecessary modifiers 'public static final' on field 'SYMBOLICNAME': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L113

Unnecessary modifiers 'public static final' on field 'PACKAGENAME': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L114

Unnecessary modifiers 'public static final' on field 'PACKAGENAMES': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L115

Unnecessary modifiers 'public static final' on field 'IMPORT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L116

Unnecessary modifiers 'public static final' on field 'IMPORTPACKAGE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L118

Unnecessary modifiers 'public static final' on field 'NAMESPACE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L120

Unnecessary modifiers 'public static final' on field 'BUNDLE_DESCRIPTION': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L121

Unnecessary modifiers 'public static final' on field 'REQUIRE_BUNDLE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L123

Unnecessary modifiers 'public static final' on field 'EXPORT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L124

Unnecessary modifiers 'public static final' on field 'EXPORT_PACKAGE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L126

Unnecessary modifiers 'public static final' on field 'SCALAR': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L127

Unnecessary modifiers 'public static final' on field 'LIST': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L128

Unnecessary modifiers 'public static final' on field 'TYPE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L129

Unnecessary modifiers 'public static final' on field 'TYPED_ATTR': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L130

Unnecessary modifiers 'public static final' on field 'REQUIREMENT': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L131

Unnecessary modifiers 'public static final' on field 'REQUIRE_CAPABILITY': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L132

Unnecessary modifiers 'public static final' on field 'CAPABILITY': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L133

Unnecessary modifiers 'public static final' on field 'PROVIDE_CAPABILITY': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L135

Unnecessary modifiers 'public static final' on field 'OBJECTCLASS': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L136

Unnecessary modifiers 'public static final' on field 'SERVICE_OR_WILDCARD': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L138

Unnecessary modifiers 'public static final' on field 'RESOURCE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L139

Unnecessary modifiers 'public static final' on field 'PREFERRED_PROVIDER': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L155

Unnecessary modifiers 'public static final' on field 'NUMBER': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L156

Unnecessary modifiers 'public static final' on field 'MAJOR': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L157

Unnecessary modifiers 'public static final' on field 'MINOR': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L158

Unnecessary modifiers 'public static final' on field 'MICRO': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L159

Unnecessary modifiers 'public static final' on field 'QUALIFIER': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L160

Unnecessary modifiers 'public static final' on field 'VERSION': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L161

Unnecessary modifiers 'public static final' on field 'ATLEAST': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L162

Unnecessary modifiers 'public static final' on field 'FLOOR': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L163

Unnecessary modifiers 'public static final' on field 'CEILING': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L164

Unnecessary modifiers 'public static final' on field 'INTERVAL': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L165

Unnecessary modifiers 'public static final' on field 'VERSIONRANGE': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L167

Unnecessary modifiers 'public static final' on field 'WILDCARD_NAME': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L168

Unnecessary modifiers 'public static final' on field 'WILDCARD_NAMES': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L169

Unnecessary modifiers 'public static final' on field 'DYNAMIC_DESCRIPTION': the field is declared in an interface type

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/Grammar.java#L170

Unnecessary modifiers 'public static final' on field 'DYNAMICIMPORT_PACKAGE': the field is declared in an interface type

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L256

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L410

Avoid empty catch blocks

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L599

Do not use `new Character(...)`, prefer `Character.valueOf(...)`

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L625

Avoid empty catch blocks

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L231

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1017

Avoid empty catch blocks

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1041

This if statement could be combined with its parent

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L201

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L214

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L227

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L232

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L499

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigAdminContentHandler.java#L68

Avoid empty catch blocks

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L113

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L114

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L115

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L116

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L117

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L118

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L119

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L120

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L123

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L124

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L125

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L126

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L127

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L128

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L129

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L130

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L141

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L307

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L360

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L440

Do not use `new Float(...)`, prefer `Float.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L445

Do not use `new Double(...)`, prefer `Double.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L460

Do not use `new Character(...)`, prefer `Character.valueOf(...)`

Best Practices UnusedPrivateMethod: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L485

Avoid unused private methods such as 'checkNext(PushbackReader, int)'.

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L734

Do not use `new Long(...)`, prefer `Long.valueOf(...)`

Best Practices PrimitiveWrapperInstantiation: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L739

Do not use `new Integer(...)`, prefer `Integer.valueOf(...)`

Code Style UselessQualifiedThis: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/Activator.java#L146

Useless qualified this usage in the same class.

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/Activator.java#L324

Avoid empty catch blocks

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/Activator.java#L326

Avoid empty catch blocks

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BasicSubsystem.java#L743

This if statement could be combined with its parent

Design SimplifiedTernary: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleEventHook.java#L191

This conditional expression can be simplified with || or &&

Error Prone AvoidBranchingStatementAsLastInLoop: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/CustomResources.java#L38

Avoid using a branching statement as the last in a loop.

Code Style UnnecessarySemicolon: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/Location.java#L40

Unnecessary semicolon

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/Location.java#L58

Avoid empty catch blocks

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/Location.java#L103

Avoid empty catch blocks

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/LockingStrategy.java#L44

Avoid empty catch blocks

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/PreferredProviderRepository.java#L42

This if statement could be combined with its parent

Design SimplifiedTernary: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/PreferredProviderRepository.java#L60

This conditional expression can be simplified with || or &&

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/RawSubsystemResource.java#L489

This if statement could be combined with its parent

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/RawSubsystemResource.java#L517

This if statement could be combined with its parent

Error Prone EmptyCatchBlock: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/RegionContextBundleHelper.java#L60

Avoid empty catch blocks

Error Prone AvoidBranchingStatementAsLastInLoop: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/RegionUpdater.java#L90

Avoid using a branching statement as the last in a loop.

Best Practices UnusedFormalParameter: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResolveContext.java#L181

Avoid unused method parameters such as 'resource'.

Best Practices UnusedFormalParameter: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResolveContext.java#L199

Avoid unused method parameters such as 'resource'.

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResourceHelper.java#L46

This if statement could be combined with its parent

Design CollapsibleIfStatements: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResourceHelper.java#L47

This if statement could be combined with its parent

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ServiceModeller.java#L26

Unnecessary modifiers 'public static' on interface 'ServiceModel': the interface is declared in an interface type

PMD (8, ubuntu-latest)

Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: jwgmeligmeyling/pmd-github-action@322e346bd76a0757c4d54ff9209e245965aa066d. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/

Code Style UselessParentheses: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmNamespaceHandler.java#L485

Useless parentheses around `(MutableComponentMetadata) component`.

Code Style UselessParentheses: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmNamespaceHandler.java#L514

Useless parentheses around `(MutableBeanMetadata) component`.

Code Style UselessParentheses: blueprint/blueprint-cm/src/main/java/org/apache/aries/blueprint/compendium/cm/CmNamespaceHandler.java#L619

Useless parentheses around `name.equals(node.getNodeName()) || name.equals(...`.

Code Style UnnecessaryFullyQualifiedName: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/BeanRecipe.java#L652

Unnecessary qualifier 'org.apache.aries.proxy': 'UnableToProxyException' is already in scope because it is imported in this file

Code Style UselessParentheses: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/GenericType.java#L198

Useless parentheses around `other.parameters == null`.

Code Style UselessParentheses: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/container/RecipeBuilder.java#L270

Useless parentheses around `beanArguments.get(0).getIndex() >= 0`.

Code Style UselessParentheses: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L600

Useless parentheses around `name.equals(node.getNodeName()) || name.equals(...`.

Code Style UnnecessaryFullyQualifiedName: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/generics/GenericsUtil.java#L588

Unnecessary qualifier 'GenericsUtil': 'resolveType' is already in scope

Code Style UnnecessaryFullyQualifiedName: blueprint/blueprint-core/src/main/java/org/apache/aries/blueprint/utils/generics/GenericsUtil.java#L590

Unnecessary qualifier 'GenericsUtil': 'getRawType' is already in scope

Code Style UselessParentheses: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/container/GenericType.java#L151

Useless parentheses around `other.parameters == null`.

Code Style UselessParentheses: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/container/NoOsgiRecipeBuilder.java#L168

Useless parentheses around `beanArguments.get(0).getIndex() >= 0`.

Code Style UselessParentheses: blueprint/blueprint-noosgi/src/main/java/org/apache/aries/blueprint/ext/impl/ExtNamespaceHandler.java#L661

Useless parentheses around `name.equals(node.getNodeName()) || name.equals(...`.

Code Style UselessParentheses: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L1434

Useless parentheses around `name.equals(node.getNodeName()) || name.equals(...`.

Code Style UnnecessaryFullyQualifiedName: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/parser/Parser.java#L1466

Unnecessary qualifier 'javax.xml': 'XMLConstants' is already in scope because it is imported in this file

Code Style UselessParentheses: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/reflect/MetadataUtil.java#L165

Useless parentheses around `arguments.get(0).getIndex() > -1`.

Code Style UselessParentheses: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/reflect/MetadataUtil.java#L167

Useless parentheses around `arguments.get(i).getIndex() > -1`.

Code Style UnnecessaryFullyQualifiedName: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/reflect/MetadataUtil.java#L176

Unnecessary qualifier 'MetadataUtil': 'BEAN_COMPARATOR' is already in scope because it is declared in an enclosing type

Code Style UselessParentheses: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/reflect/MetadataUtil.java#L194

Useless parentheses around `BeanMetadata.SCOPE_PROTOTYPE.equals(metadata.ge...`.

Code Style UselessParentheses: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/reflect/MetadataUtil.java#L199

Useless parentheses around `BeanMetadata.SCOPE_SINGLETON.equals(metadata.ge...`.

Code Style UselessParentheses: blueprint/blueprint-parser/src/main/java/org/apache/aries/blueprint/reflect/MetadataUtil.java#L204

Useless parentheses around `metadata.getScope() != null && !BeanMetadata.SC...`.

Code Style UnnecessaryFullyQualifiedName: blueprint-maven-plugin/blueprint-maven-plugin/target/generated-sources/plugin/org/apache/aries/blueprint/blueprint_maven_plugin/HelpMojo.java#L43

Unnecessary qualifier 'java.lang': 'String' is already in scope because it is declared in java.lang

Code Style UselessParentheses: esa-ant-task/src/main/java/org/apache/aries/ant/taskdefs/EsaTask.java#L273

Useless parentheses around `subsystemContentHeader.length() - 1`.

Code Style UnnecessaryFullyQualifiedName: esa-maven-plugin/src/main/java/org/apache/aries/plugin/esa/EsaMojo.java#L342

Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/BlueprintMetadata.java#L102

Unnecessary qualifier 'BlueprintMetadataMBean': 'SERVICE_METADATA' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/BlueprintMetadata.java#L104

Unnecessary qualifier 'BlueprintMetadataMBean': 'BEAN_METADATA' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/BlueprintMetadata.java#L106

Unnecessary qualifier 'BlueprintMetadataMBean': 'SERVICE_REFERENCE_METADATA' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/BlueprintState.java#L86

Unnecessary qualifier 'BlueprintStateMBean': 'OSGI_BLUEPRINT_EVENTS_TYPE' is already in scope because it is inherited by an enclosing type

Code Style UselessParentheses: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/RegistrableStandardEmitterMBean.java#L53

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/RegistrableStandardEmitterMBean.java#L113

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/RegistrableStandardEmitterMBean.java#L125

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/RegistrableStandardEmitterMBean.java#L136

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-blueprint-core/src/main/java/org/apache/aries/jmx/blueprint/impl/RegistrableStandardEmitterMBean.java#L149

Useless parentheses around `impl`.

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/cm/ConfigurationAdminMBeanHandler.java#L54

Unnecessary qualifier 'org.apache.aries.jmx.cm': 'ConfigurationAdmin' is already in scope because it is declared in the same package

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/BundleData.java#L233

Unnecessary qualifier 'BundleStateMBean': 'BUNDLE_TYPE' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/BundleWiringData.java#L122

Unnecessary qualifier 'BundleWiringData': 'getCapReqCompositeData' is already in scope

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/BundleWiringData.java#L140

Unnecessary qualifier 'BundleWiringData': 'getCapReqCompositeData' is already in scope

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/codec/ServiceData.java#L102

Unnecessary qualifier 'ServiceStateMBean': 'SERVICE_TYPE' is already in scope because it is imported in this file

Code Style UselessParentheses: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/BundleState.java#L268

Useless parentheses around `PackageAdmin.BUNDLE_TYPE_FRAGMENT == packageAdm...`.

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/BundleState.java#L330

Unnecessary qualifier 'BundleStateMBean': 'BUNDLE_TYPE' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/ServiceState.java#L156

Unnecessary qualifier 'ServiceStateMBean': 'SERVICE_TYPE' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/wiring/BundleWiringState.java#L86

Unnecessary qualifier 'BundleWiringStateMBean': 'BUNDLES_WIRING_TYPE' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/wiring/BundleWiringState.java#L153

Unnecessary qualifier 'BundleWiringStateMBean': 'REVISIONS_REQUIREMENTS_TYPE' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/wiring/BundleWiringState.java#L169

Unnecessary qualifier 'BundleWiringStateMBean': 'REVISIONS_CAPABILITIES_TYPE' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/wiring/BundleWiringState.java#L185

Unnecessary qualifier 'BundleWiringStateMBean': 'BUNDLES_WIRING_TYPE' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: jmx/jmx-core/src/main/java/org/apache/aries/jmx/framework/wiring/BundleWiringState.java#L210

Unnecessary qualifier 'BundleWiringStateMBean': 'BUNDLES_WIRING_TYPE' is already in scope because it is inherited by an enclosing type

Code Style UselessParentheses: jmx/jmx-core/src/main/java/org/apache/aries/jmx/util/shared/RegistrableStandardEmitterMBean.java#L53

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-core/src/main/java/org/apache/aries/jmx/util/shared/RegistrableStandardEmitterMBean.java#L113

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-core/src/main/java/org/apache/aries/jmx/util/shared/RegistrableStandardEmitterMBean.java#L125

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-core/src/main/java/org/apache/aries/jmx/util/shared/RegistrableStandardEmitterMBean.java#L136

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-core/src/main/java/org/apache/aries/jmx/util/shared/RegistrableStandardEmitterMBean.java#L149

Useless parentheses around `impl`.

Code Style UselessParentheses: jmx/jmx-whiteboard/src/main/java/org/apache/aries/jmx/whiteboard/MBeanHolder.java#L132

Useless parentheses around `(RuntimeException) e`.

Code Style UselessParentheses: jndi/jndi-core/src/main/java/org/apache/aries/jndi/ContextProvider.java#L37

Useless parentheses around `reference.getBundle() != null`.

Code Style UnnecessaryFullyQualifiedName: jndi/jndi-core/src/main/java/org/apache/aries/jndi/ObjectFactoryHelper.java#L167

Unnecessary qualifier 'ObjectFactoryHelper': 'findObjectFactoryByClassName' is already in scope

Code Style UnnecessaryFullyQualifiedName: jndi/jndi-core/src/main/java/org/apache/aries/jndi/startup/Activator.java#L128

Unnecessary qualifier 'Activator': 'getService' is already in scope

Code Style UselessParentheses: jndi/jndi-url/src/main/java/org/apache/aries/jndi/services/ServiceHelper.java#L397

Useless parentheses around `otherKey.requesting == requesting && otherKey.s...`.

Code Style UselessParentheses: proxy/proxy-api/src/main/java/org/apache/aries/proxy/FinalModifierException.java#L44

Useless parentheses around `finalMethods == null || finalMethods.equals("")`.

Code Style UselessParentheses: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/AbstractProxyManager.java#L110

Useless parentheses around `proxy != null && ((proxy instanceof WovenProxy ...`.

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L200

Unnecessary qualifier 'Opcodes': 'V1_6' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyAdapter.java#L560

Unnecessary qualifier 'String': 'format' is already in scope

Code Style UnnecessaryImport: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/common/AbstractWovenProxyMethodAdapter.java#L32

Unused static import 'org.objectweb.asm.Opcodes.ASM9'

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L168

Unnecessary qualifier 'Opcodes': 'ICONST_0' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L204

Unnecessary qualifier 'Opcodes': 'ACONST_NULL' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L500

Unnecessary qualifier 'java.lang': 'Boolean' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L503

Unnecessary qualifier 'java.lang': 'Byte' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L506

Unnecessary qualifier 'java.lang': 'Character' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L509

Unnecessary qualifier 'java.lang': 'Double' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L512

Unnecessary qualifier 'java.lang': 'Float' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L515

Unnecessary qualifier 'java.lang': 'Integer' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L518

Unnecessary qualifier 'java.lang': 'Long' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L521

Unnecessary qualifier 'java.lang': 'Short' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L534

Unnecessary qualifier 'java.lang': 'Class' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassAdapter.java#L558

Unnecessary qualifier 'java.lang': 'Object' is already in scope because it is declared in java.lang

Code Style UnnecessaryImport: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassGenerator.java#L24

Unused import 'java.lang.reflect.Constructor'

Code Style UselessParentheses: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/gen/ProxySubclassGenerator.java#L306

Useless parentheses around `proxies != null && proxies.containsValue(aClass...`.

Code Style UselessParentheses: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/InterfaceCombiningClassAdapter.java#L96

Useless parentheses around `(access & (ACC_PRIVATE | ACC_SYNTHETIC)) == (AC...`.

Code Style UselessParentheses: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/InterfaceCombiningClassAdapter.java#L99

Useless parentheses around `(access & ACC_STATIC) == ACC_STATIC`.

Code Style UselessParentheses: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/InterfaceCombiningClassAdapter.java#L99

Useless parentheses around `ACC_STATIC`.

Code Style UselessParentheses: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/interfaces/InterfaceCombiningClassAdapter.java#L99

Useless parentheses around `ACC_STATIC`.

Code Style UnnecessaryFullyQualifiedName: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/weaving/ProxyWeavingHook.java#L117

Unnecessary qualifier 'String': 'format' is already in scope

Code Style UnnecessaryImport: proxy/proxy-impl/src/main/java/org/apache/aries/proxy/impl/weaving/SyntheticSerialVerUIDAdder.java#L21

Unused static import 'org.objectweb.asm.Opcodes.ASM9'

Code Style UnnecessaryFullyQualifiedName: pushstream/pushstream/src/main/java/org/apache/aries/pushstream/AbstractPushStreamImpl.java#L179

Unnecessary qualifier 'State': 'CLOSED' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: pushstream/pushstream/src/main/java/org/apache/aries/pushstream/AbstractPushStreamImpl.java#L191

Unnecessary qualifier 'State': 'CLOSED' is already in scope because it is imported in this file

Code Style UselessParentheses: pushstream/pushstream/src/main/java/org/osgi/util/pushstream/PushbackPolicyOption.java#L81

Useless parentheses around `value * used`.

Code Style UnnecessaryFullyQualifiedName: quiesce/quiesce-manager/src/main/java/org/apache/aries/quiesce/manager/impl/QuiesceManagerImpl.java#L314

Unnecessary qualifier 'QuiesceManagerImpl': 'stillQuiescing' is already in scope

Code Style UnnecessaryFullyQualifiedName: quiesce/quiesce-manager/src/main/java/org/apache/aries/quiesce/manager/impl/QuiesceManagerImpl.java#L317

Unnecessary qualifier 'QuiesceManagerImpl': 'stopBundle' is already in scope

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-beans/src/main/java/org/apache/aries/samples/ariestrader/beans/AccountDataBeanImpl.java#L110

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-beans/src/main/java/org/apache/aries/samples/ariestrader/beans/AccountDataBeanImpl.java#L111

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-beans/src/main/java/org/apache/aries/samples/ariestrader/beans/HoldingDataBeanImpl.java#L87

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-beans/src/main/java/org/apache/aries/samples/ariestrader/beans/OrderDataBeanImpl.java#L118

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-beans/src/main/java/org/apache/aries/samples/ariestrader/beans/OrderDataBeanImpl.java#L119

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/AccountDataBeanImpl.java#L153

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/AccountDataBeanImpl.java#L154

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/HoldingDataBeanImpl.java#L104

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/OrderDataBeanImpl.java#L149

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/OrderDataBeanImpl.java#L150

Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-entities/src/main/java/org/apache/aries/samples/ariestrader/entities/QuoteDataBeanImpl.java#L49

Unnecessary qualifier 'org.apache.aries.samples.ariestrader.entities': 'QuoteDataBeanImpl' is already in scope because it is declared in the same package

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L230

Useless parentheses around `new BigDecimal(quantity).multiply(price)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L301

Useless parentheses around `new BigDecimal(quantity).multiply(price)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L318

Useless parentheses around `orderData.getOrderStatus().equalsIgnoreCase("ca...`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jdbc/src/main/java/org/apache/aries/samples/ariestrader/persist/jdbc/TradeJdbc.java#L570

Useless parentheses around `rs.next()`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L194

Useless parentheses around `new BigDecimal(quantity).multiply(price)`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L274

Unnecessary qualifier 'java.sql': 'Timestamp' is already in scope because it is imported in this file

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L280

Useless parentheses around `new BigDecimal(quantity).multiply(price)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L309

Useless parentheses around `order.getOrderStatus().equalsIgnoreCase("cancel...`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L391

Unnecessary qualifier 'java.sql': 'Timestamp' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L516

Unnecessary qualifier 'org.apache.aries.samples.ariestrader.entities': 'AccountDataBeanImpl' is already in scope because it is imported in this file

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jpa-am/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/am/TradeJpaAm.java#L647

Useless parentheses around `newPrice.subtract(quote.getOpen()).doubleValue()`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L176

Useless parentheses around `new BigDecimal(quantity).multiply(price)`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L233

Unnecessary qualifier 'java.sql': 'Timestamp' is already in scope because it is imported in this file

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L239

Useless parentheses around `new BigDecimal(quantity).multiply(price)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L257

Useless parentheses around `order.getOrderStatus().equalsIgnoreCase("cancel...`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L335

Unnecessary qualifier 'java.sql': 'Timestamp' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L416

Unnecessary qualifier 'org.apache.aries.samples.ariestrader.entities': 'AccountDataBeanImpl' is already in scope because it is imported in this file

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-persist-jpa-cm/src/main/java/org/apache/aries/samples/ariestrader/persist/jpa/cm/TradeJpaCm.java#L503

Useless parentheses around `newPrice.subtract(quote.getOpen()).doubleValue()`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/FinancialUtils.java#L26

Useless parentheses around `new BigDecimal(0.00)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/FinancialUtils.java#L27

Useless parentheses around `new BigDecimal(1.00)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/FinancialUtils.java#L28

Useless parentheses around `new BigDecimal(100.00)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/KeyBlock.java#L53

Useless parentheses around `max - min`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/KeyBlock.java#L77

Unnecessary qualifier 'java.lang': 'RuntimeException' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L263

Unnecessary qualifier 'java.lang': 'String' is already in scope because it is declared in java.lang

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L368

Useless parentheses around `new Float(random() * i)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L371

Useless parentheses around `new Float(random() * i)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L374

Useless parentheses around `new BigDecimal(random() * f)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L396

Useless parentheses around `new Integer(rndInt(200)).floatValue()`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L396

Useless parentheses around `new Integer(rndInt(200))`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L407

Useless parentheses around `new BigDecimal(percentGain)`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L415

Useless parentheses around `new Integer(rndInt(200)).floatValue()`.

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L415

Useless parentheses around `new Integer(rndInt(200))`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L467

Unnecessary qualifier 'java.lang': 'String' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L868

Unnecessary qualifier 'TradeConfig': 'marketSummaryInterval' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L872

Unnecessary qualifier 'TradeConfig': 'marketSummaryInterval' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-util/src/main/java/org/apache/aries/samples/ariestrader/util/TradeConfig.java#L880

Unnecessary qualifier 'java.lang': 'String' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeAppServlet.java#L64

Unnecessary qualifier 'java.lang': 'String' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeAppServlet.java#L76

Unnecessary qualifier 'javax.servlet.http': 'HttpServletRequest' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeAppServlet.java#L77

Unnecessary qualifier 'javax.servlet.http': 'HttpServletResponse' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeAppServlet.java#L90

Unnecessary qualifier 'javax.servlet.http': 'HttpServletRequest' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeAppServlet.java#L91

Unnecessary qualifier 'javax.servlet.http': 'HttpServletResponse' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeBuildDB.java#L172

Unnecessary qualifier 'java.math': 'BigDecimal' is already in scope because it is imported in this file

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeBuildDB.java#L203

Useless parentheses around `TradeConfig.rndInt(100000)`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeScenarioServlet.java#L63

Unnecessary qualifier 'java.lang': 'String' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeScenarioServlet.java#L76

Unnecessary qualifier 'javax.servlet.http': 'HttpServletRequest' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeScenarioServlet.java#L76

Unnecessary qualifier 'javax.servlet.http': 'HttpServletResponse' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeScenarioServlet.java#L88

Unnecessary qualifier 'javax.servlet.http': 'HttpServletRequest' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeScenarioServlet.java#L88

Unnecessary qualifier 'javax.servlet.http': 'HttpServletResponse' is already in scope because it is imported in this file

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeScenarioServlet.java#L256

Useless parentheses around `holdingData.getPurchaseDate().equals(new Date(0))`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L83

Unnecessary qualifier 'javax.servlet': 'ServletException' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L83

Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L89

Useless parentheses around `TradeConfig.getLongRun() ? new ArrayList() : (A...`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L97

Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L153

Unnecessary qualifier 'javax.servlet': 'ServletException' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L153

Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L174

Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L224

Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L267

Unnecessary qualifier 'javax.servlet': 'ServletException' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L267

Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L284

Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L326

Unnecessary qualifier 'javax.servlet': 'ServletException' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L326

Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L352

Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L407

Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L498

Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L595

Useless parentheses around `passwd.equals(cpasswd)`.

Code Style UnnecessaryFullyQualifiedName: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/TradeServletAction.java#L662

Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang

Code Style UselessParentheses: samples/ariestrader/modules/ariestrader-web/src/main/java/org/apache/aries/samples/ariestrader/web/prims/PingSession3.java#L135

Useless parentheses around `NUM_OBJECTS * 1024`.

Code Style UselessParentheses: samples/blog/blog-persistence-jdbc/src/main/java/org/apache/aries/samples/blog/persistence/jdbc/BlogPersistenceServiceImpl.java#L352

Useless parentheses around `blogs.subList(firstPostIndex, (lastIndex > maxi...`.

Code Style UnnecessaryFullyQualifiedName: spi-fly/spi-fly-weaver/src/main/java/org/apache/aries/spifly/weaver/TCCLSetterVisitor.java#L68

Unnecessary qualifier 'Opcodes': 'ASM9' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: spi-fly/spi-fly-weaver/src/main/java/org/apache/aries/spifly/weaver/TCCLSetterVisitor.java#L162

Unnecessary qualifier 'Opcodes': 'ASM7' is already in scope because it is inherited by an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/SubsystemPermission.java#L539

Unnecessary qualifier 'java.io': 'ObjectOutputStream' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/SubsystemPermission.java#L554

Unnecessary qualifier 'java.io': 'ObjectInputStream' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-api/src/main/java/org/osgi/service/subsystem/SubsystemPermission.java#L745

Unnecessary qualifier 'java.io': 'ObjectInputStream' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/DeploymentManifest.java#L55

Unnecessary qualifier 'DeploymentManifest': 'DEPLOYED_CONTENT' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/DeploymentManifest.java#L174

Unnecessary qualifier 'java.util.jar': 'Manifest' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/SubsystemManifest.java#L120

Unnecessary qualifier 'java.util.jar': 'Manifest' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/archive/SubsystemManifest.java#L150

Unnecessary qualifier 'java.util.jar': 'Manifest' is already in scope because it is imported in this file

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L81

Useless parentheses around `caseSensitive`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L186

Useless parentheses around `obeyMandatory`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L453

Useless parentheses around `((Comparable) lhs).compareTo(rhs) == 0`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L462

Useless parentheses around `((Comparable) lhs).compareTo(rhs) >= 0`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L471

Useless parentheses around `((Comparable) lhs).compareTo(rhs) <= 0`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L505

Useless parentheses around `lhs.equals(rhs)`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L562

Useless parentheses around `(Character) lhs`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/CapabilitySet.java#L563

Useless parentheses around `(Character) rhs`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1259

Useless parentheses around `allServices`.

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1276

Unnecessary qualifier 'org.osgi.framework.hooks.service': 'ListenerHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1300

Unnecessary qualifier 'org.osgi.framework.hooks.service': 'ListenerHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1373

Unnecessary qualifier 'org.osgi.framework.hooks.resolver': 'ResolverHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1397

Unnecessary qualifier 'org.osgi.framework.hooks.resolver': 'ResolverHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1422

Unnecessary qualifier 'org.osgi.framework.hooks.resolver': 'ResolverHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1447

Unnecessary qualifier 'org.osgi.framework.hooks.resolver': 'ResolverHook' is already in scope because it is imported in this file

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1737

Useless parentheses around `(Class) arg1`.

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1764

Unnecessary qualifier 'org.osgi.framework.hooks.service': 'ListenerHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1768

Unnecessary qualifier 'org.osgi.framework.hooks.service': 'ListenerHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1780

Unnecessary qualifier 'org.osgi.framework.hooks.resolver': 'ResolverHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1784

Unnecessary qualifier 'org.osgi.framework.hooks.resolver': 'ResolverHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1790

Unnecessary qualifier 'org.osgi.framework.hooks.resolver': 'ResolverHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SecureAction.java#L1795

Unnecessary qualifier 'org.osgi.framework.hooks.resolver': 'ResolverHook' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L197

Unnecessary qualifier 'SimpleFilter': 'AND' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L210

Unnecessary qualifier 'SimpleFilter': 'OR' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L223

Unnecessary qualifier 'SimpleFilter': 'NOT' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L252

Unnecessary qualifier 'SimpleFilter': 'subfilter' is already in scope

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L256

Unnecessary qualifier 'SimpleFilter': 'subfilter' is already in scope

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L578

Unnecessary qualifier 'SimpleFilter': 'GTE' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L583

Unnecessary qualifier 'SimpleFilter': 'NOT' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L588

Unnecessary qualifier 'SimpleFilter': 'LTE' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L600

Unnecessary qualifier 'SimpleFilter': 'LTE' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L605

Unnecessary qualifier 'SimpleFilter': 'NOT' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L610

Unnecessary qualifier 'SimpleFilter': 'GTE' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L617

Unnecessary qualifier 'SimpleFilter': 'parseSubstring' is already in scope

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L624

Unnecessary qualifier 'SimpleFilter': 'SUBSTRING' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L632

Unnecessary qualifier 'SimpleFilter': 'EQ' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L645

Unnecessary qualifier 'SimpleFilter': 'AND' is already in scope because it is declared in an enclosing type

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/capabilityset/SimpleFilter.java#L649

Unnecessary qualifier 'SimpleFilter': 'MATCH_ALL' is already in scope because it is declared in an enclosing type

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L570

Useless parentheses around `token = c`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L579

Useless parentheses around `token = TOKEN_NAME`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L585

Useless parentheses around `token = c`.

Code Style UselessParentheses: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/content/ConfigurationHandler.java#L589

Useless parentheses around `token = -1`.

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleResource.java#L178

Unnecessary qualifier 'org.osgi.framework': 'Constants' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleResource.java#L195

Unnecessary qualifier 'org.osgi.framework': 'Constants' is already in scope because it is imported in this file

Code Style UnnecessaryFullyQualifiedName: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/PreferredProviderRepository.java#L30

Unnecessary qualifier 'org.apache.aries.subsystem.core.repository': 'Repository' is already in scope because it is imported in this file

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bump org.apache.ant:ant from 1.10.16 to 1.10.17 #490

Summary