Pull Request Labeler

ARIES-2165: Run blueprint build on java 11, 17, 21 and 25 #447

Triggered via pull request February 1, 2026 16:30

alien11689

opened #785

Status Success

Total duration 8s

Artifacts –

labeler.yml

on: pull_request_target

labeler

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

Code Style UnnecessaryModifier: subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/StartAction.java#L60

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

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

ARIES-2165: Run blueprint build on java 11, 17, 21 and 25 #447

Summary