| Violation | Line |
|---|---|
| DC: Possible doublecheck on org.mevenide.ui.eclipse.Mevenide.resourceBundle in org.mevenide.ui.eclipse.Mevenide.getResourceBundle() | 180 |
| ST: Write to static field org.mevenide.ui.eclipse.Mevenide.plugin from instance method org.mevenide.ui.eclipse.Mevenide.<init>() | 80 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.Mevenide.mavenClasspathManager | 0 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.Mevenide.pomManager | 0 |
| CD: Class org.mevenide.ui.eclipse.Mevenide has a circular dependency with other classes. | 0 |
| CD: Class org.mevenide.ui.eclipse.Mevenide has a circular dependency with other classes. | 0 |
| CD: Class org.mevenide.ui.eclipse.Mevenide has a circular dependency with other classes. | 0 |
| Violation | Line |
|---|---|
| SA: Self assignment of local variable in org.mevenide.ui.eclipse.actions.CreatePomAction.run(org.eclipse.jface.action.IAction) | 53 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.nature.ActionDefinitions defines equals and uses Object.hashCode() | 0 |
| RCN: Redundant nullcheck of value known to be non-null org.mevenide.ui.eclipse.nature.ActionDefinitions.isEnabled(org.eclipse.core.resources.IProject) | 52 |
| RCN: Redundant nullcheck of value known to be non-null org.mevenide.ui.eclipse.nature.ActionDefinitions.isEnabled(org.eclipse.core.resources.IProject) | 56 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.nature.ActionDefinitions.configuration | 0 |
| Violation | Line |
|---|---|
| SF: Switch statement found in org.mevenide.ui.eclipse.nature.PatternsTab$4$1.handleEvent(org.eclipse.swt.widgets.Event) where one case falls thru to the next case | 196 |
| Violation | Line |
|---|---|
| WMI: Method org.mevenide.ui.eclipse.nature.PatternsTab.initPatterns(org.eclipse.debug.core.ILaunchConfiguration) makes inefficient use of keySet iterator instead of entrySet iterator | 263 |
| Violation | Line |
|---|---|
| SBSC: Method org.mevenide.ui.eclipse.preferences.DependencyTypeRegistry.storeTypes(java.util.List) concatenates strings using + in a loop | 85 |
| MS: org.mevenide.ui.eclipse.preferences.DependencyTypeRegistry.KNOWN_TYPES is a mutable array | 0 |
| Violation | Line |
|---|---|
| RV: org.mevenide.ui.eclipse.preferences.PreferencesManager.loadPreferences() ignores return value of java.io.File.createNewFile() | 57 |
| Violation | Line |
|---|---|
| OS: org.mevenide.ui.eclipse.util.FileUtils.copyFile(java.io.File,java.io.File) may fail to close stream | 84 |
| OS: org.mevenide.ui.eclipse.util.FileUtils.copyFile(java.io.File,java.io.File) may fail to close stream | 85 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.util.JavaProjectUtils.findExclusionPatterns(String,org.eclipse.core.resources.IProject) return a zero length array rather than null? | 226 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.wizard.MavenProjectWizardBasicSettingsPage$LocationGroup.<init>(MavenProjectWizardBasicSettingsPage,org.eclipse.swt.widgets.Composite) | 135 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.wizard.MavenProjectWizardBasicSettingsPage$NameGroup.<init>(MavenProjectWizardBasicSettingsPage,org.eclipse.swt.widgets.Composite,String) | 74 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.wizard.MavenProjectWizardBasicSettingsPage$TemplateGroup.<init>(MavenProjectWizardBasicSettingsPage,org.eclipse.swt.widgets.Composite) | 257 |
| SIC: The class org.mevenide.ui.eclipse.wizard.MavenProjectWizardBasicSettingsPage$TemplateGroup could be refactored into a _static_ inner class | 0 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.wizard.MavenProjectWizardBasicSettingsPOMPage$OrganizationGroup.<init>(MavenProjectWizardBasicSettingsPOMPage,org.eclipse.swt.widgets.Composite) | 166 |
| SIC: The class org.mevenide.ui.eclipse.wizard.MavenProjectWizardBasicSettingsPOMPage$OrganizationGroup could be refactored into a _static_ inner class | 0 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.wizard.MavenProjectWizardBasicSettingsPOMPage$ProjectGroup.<init>(MavenProjectWizardBasicSettingsPOMPage,org.eclipse.swt.widgets.Composite) | 67 |
| SIC: The class org.mevenide.ui.eclipse.wizard.MavenProjectWizardBasicSettingsPOMPage$ProjectGroup could be refactored into a _static_ inner class | 0 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.wizard.MavenProjectWizardFirstPage$LocationGroup.<init>(MavenProjectWizardFirstPage,org.eclipse.swt.widgets.Composite) | 126 |
| Violation | Line |
|---|---|
| MF: Class org.mevenide.ui.eclipse.wizard.NewPomWizard defines field that obscures superclass field org.eclipse.ui.wizards.newresource.BasicNewResourceWizard.selection | 0 |
| RI: Class org.mevenide.ui.eclipse.wizard.NewPomWizard implements same interface as superclass. | 0 |
| Violation | Line |
|---|---|
| NP: Load of known null value in org.mevenide.ui.eclipse.wizard.NewPomWizardFirstPage.createFile(org.eclipse.core.resources.IFile,java.io.InputStream,org.eclipse.core.runtime.IProgressMonitor) | 261 |
| BC: Unchecked/unconfirmed cast from java.lang.Throwable to class org.eclipse.core.runtime.CoreException in org.mevenide.ui.eclipse.wizard.NewPomWizardFirstPage.createNewFile() | 215 |
| UwF: Unwritten field: org.mevenide.ui.eclipse.wizard.NewPomWizardFirstPage.initialContainerFullPath | 0 |
| UwF: Unwritten field: org.mevenide.ui.eclipse.wizard.NewPomWizardFirstPage.initialFileName | 0 |
| CD: Class org.mevenide.ui.eclipse.wizard.NewPomWizardFirstPage has a circular dependency with other classes. | 0 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.adapters.properties.AbstractPomPropertySource.getChildren(Object) return a zero length array rather than null? | 62 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.BranchPropertySource.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.BranchPropertySource.descriptors | 56 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.ContributorPropertySource.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.ContributorPropertySource.descriptors | 88 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.DependencyPropertySource.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.DependencyPropertySource.descriptors | 105 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.LicensePropertySource.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.LicensePropertySource.descriptors | 87 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.MailingListPropertySource.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.MailingListPropertySource.descriptors | 71 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.PropertyProxy.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.PropertyProxy.descriptors | 70 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.ResourcePropertySource.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.ResourcePropertySource.descriptors | 87 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.SourceModificationPropertySource.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.SourceModificationPropertySource.descriptors | 59 |
| Violation | Line |
|---|---|
| EI: org.mevenide.ui.eclipse.adapters.properties.VersionPropertySource.getPropertyDescriptors() may expose internal representation by returning org.mevenide.ui.eclipse.adapters.properties.VersionPropertySource.descriptors | 66 |
| Violation | Line |
|---|---|
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.source.SourceViewerConfiguration to class org.mevenide.ui.eclipse.editors.jelly.XMLConfiguration in org.mevenide.ui.eclipse.editors.jelly.MavenXmlEditor.performSave(boolean,org.eclipse.core.runtime.IProgressMonitor) | 57 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.editors.jelly.MavenXmlEditor.outlinePage | 0 |
| Violation | Line |
|---|---|
| SA: Self assignment of local variable in org.mevenide.ui.eclipse.editors.jelly.Namespace.getCandidateTags(String) | 157 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.editors.jelly.TagRule.evaluate(org.eclipse.jface.text.rules.ICharacterScanner) | 91 |
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.editors.jelly.TagRule.evaluate(org.eclipse.jface.text.rules.ICharacterScanner) | 85 |
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.editors.jelly.TagRule.evaluate(org.eclipse.jface.text.rules.ICharacterScanner) | 72 |
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.editors.jelly.TagRule.evaluate(org.eclipse.jface.text.rules.ICharacterScanner) | 58 |
| Violation | Line |
|---|---|
| EI2: org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.<init>(org.eclipse.jface.text.rules.IPartitionTokenScanner,String[]) may expose internal representation by storing an externally mutable object into org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.fLegalContentTypes | 37 |
| EI: org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.getLegalContentTypes() may expose internal representation by returning org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.fLegalContentTypes | 319 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.documentChanged2(org.eclipse.jface.text.DocumentEvent) | 128 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.documentChanged2(org.eclipse.jface.text.DocumentEvent) | 200 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.documentChanged2(org.eclipse.jface.text.DocumentEvent) | 169 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.findClosestPosition(int) | 223 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.findClosestPosition(int) | 220 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.getPartition(int) | 246 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.getPartition(int) | 258 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.getPartition(int) | 251 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.eclipse.jface.text.TypedPosition in org.mevenide.ui.eclipse.editors.jelly.XMLDocumentPartitioner.computePartitioning(int,int) | 281 |
| Violation | Line |
|---|---|
| RCN: Redundant nullcheck of value known to be non-null org.mevenide.ui.eclipse.editors.jelly.XMLDocumentProvider.createDocument(Object) | 32 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.editors.jelly.XMLNode.getChildren(Object) return a zero length array rather than null? | 145 |
| NP: Possible null pointer dereference in org.mevenide.ui.eclipse.editors.jelly.XMLNode.getAttributeName() on exception path | 294 |
| NP: Possible null pointer dereference in org.mevenide.ui.eclipse.editors.jelly.XMLNode.getAttributeValue() on exception path | 332 |
| UCF: Useless control flow in org.mevenide.ui.eclipse.editors.jelly.XMLNode.getValue() | 270 |
| ITA: Method org.mevenide.ui.eclipse.editors.jelly.XMLNode.getChildren(Object) uses Collection.toArray() with zero-length array argument | 143 |
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.editors.jelly.XMLNode.containsOnlyWhitespaces() | 149 |
| Violation | Line |
|---|---|
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.XMLPositionUpdater.adaptToReplace() | 29 |
| Violation | Line |
|---|---|
| RCN: Nullcheck of value previously dereferenced org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.fix(org.eclipse.jface.text.Position[],int,XMLNode) | 246 |
| RCN: Nullcheck of value previously dereferenced org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.fix(org.eclipse.jface.text.Position[],int,XMLNode) | 243 |
| RCN: Nullcheck of value previously dereferenced org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.fix(org.eclipse.jface.text.Position[],int,XMLNode) | 276 |
| RCN: Nullcheck of value previously dereferenced org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.fix(org.eclipse.jface.text.Position[],int,XMLNode) | 273 |
| UCF: Useless control flow in org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.documentChanged(org.eclipse.jface.text.DocumentEvent) | 166 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.IRegion to class org.eclipse.jface.text.TypedRegion in org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.reconcile(org.eclipse.jface.text.IRegion) | 62 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.createTree(org.eclipse.jface.text.IDocument) | 87 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.documentChanged(org.eclipse.jface.text.DocumentEvent) | 134 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.documentChanged(org.eclipse.jface.text.DocumentEvent) | 146 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.fix(org.eclipse.jface.text.Position[],int,XMLNode) | 200 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.document | 0 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.editors.jelly.XMLReconciler.storedPos | 0 |
| Violation | Line |
|---|---|
| RI: Class org.mevenide.ui.eclipse.editors.pom.PomEditorSelectionProvider implements same interface as superclass. | 0 |
| Violation | Line |
|---|---|
| IP: A parameter to org.mevenide.ui.eclipse.editors.pom.PomXmlSourcePage.init(org.eclipse.ui.IEditorSite,org.eclipse.ui.IEditorInput) is dead upon entry but overwritten | 80 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.goals.model.Element defines equals and uses Object.hashCode() | 0 |
| EI: org.mevenide.ui.eclipse.goals.model.Element.getPrereqs() may expose internal representation by returning org.mevenide.ui.eclipse.goals.model.Element.prereqs | 47 |
| EI2: org.mevenide.ui.eclipse.goals.model.Element.setPrereqs(Element[]) may expose internal representation by storing an externally mutable object into org.mevenide.ui.eclipse.goals.model.Element.prereqs | 51 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.goals.model.GoalsProvider.getChildren(Object) return a zero length array rather than null? | 79 |
| NP: Possible null pointer dereference in org.mevenide.ui.eclipse.goals.model.GoalsProvider.<init>(org.mevenide.context.IQueryContext) on exception path | 52 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.goals.model.Plugin defines equals and uses Object.hashCode() | 0 |
| Violation | Line |
|---|---|
| EI2: org.mevenide.ui.eclipse.goals.view.GoalsOrderDialog.<init>(org.eclipse.swt.widgets.Shell,Object[]) may expose internal representation by storing an externally mutable object into org.mevenide.ui.eclipse.goals.view.GoalsOrderDialog.fTargets | 57 |
| Violation | Line |
|---|---|
| SBSC: Method org.mevenide.ui.eclipse.goals.view.GoalsPickerDialog$3.widgetSelected(org.eclipse.swt.events.SelectionEvent) concatenates strings using + in a loop | 276 |
| Violation | Line |
|---|---|
| SBSC: Method org.mevenide.ui.eclipse.goals.view.GoalsPickerDialog.updateCheckedItems(org.eclipse.jface.viewers.CheckStateChangedEvent) concatenates strings using + in a loop | 324 |
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.goals.view.GoalsPickerDialog.updateStyledTextWidgetHyperlink(org.eclipse.swt.events.SelectionEvent) | 349 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.goals.view.GoalsPickerDialog.goalsViewer | 0 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.goals.view.GoalsPickerDialog.pluginHomeURLText | 0 |
| CD: Class org.mevenide.ui.eclipse.goals.view.GoalsPickerDialog has a circular dependency with other classes. | 0 |
| Violation | Line |
|---|---|
| Dm: org.mevenide.ui.eclipse.launch.configuration.MavenArgumentsTab$1.widgetSelected(org.eclipse.swt.events.SelectionEvent) invokes dubious Boolean constructor; use Boolean.valueOf(...) instead | 170 |
| Violation | Line |
|---|---|
| SF: Switch statement found in org.mevenide.ui.eclipse.launch.configuration.MavenArgumentsTab$6$1.handleEvent(org.eclipse.swt.widgets.Event) where one case falls thru to the next case | 330 |
| Violation | Line |
|---|---|
| Dm: org.mevenide.ui.eclipse.launch.configuration.MavenArgumentsTab.initOptionsMap(org.eclipse.debug.core.ILaunchConfiguration) invokes dubious Boolean constructor; use Boolean.valueOf(...) instead | 440 |
| SA: Self assignment of local variable in org.mevenide.ui.eclipse.launch.configuration.MavenArgumentsTab.initOptionsMap(org.eclipse.debug.core.ILaunchConfiguration) | 435 |
| WMI: Method org.mevenide.ui.eclipse.launch.configuration.MavenArgumentsTab.initOptionsMap(org.eclipse.debug.core.ILaunchConfiguration) makes inefficient use of keySet iterator instead of entrySet iterator | 431 |
| Violation | Line |
|---|---|
| NP: Possible null pointer dereference in org.mevenide.ui.eclipse.launch.configuration.MavenLaunchDelegate.getDynamicProperties(org.eclipse.debug.core.ILaunchConfiguration,java.util.List) on exception path | 201 |
| SA: Self assignment of local variable in org.mevenide.ui.eclipse.launch.configuration.MavenLaunchDelegate.getToolsJar() | 382 |
| WMI: Method org.mevenide.ui.eclipse.launch.configuration.MavenLaunchDelegate.launch(org.eclipse.debug.core.ILaunchConfiguration,String,org.eclipse.debug.core.ILaunch,org.eclipse.core.runtime.IProgressMonitor) makes inefficient use of keySet iterator instead of entrySet iterator | 111 |
| WMI: Method org.mevenide.ui.eclipse.launch.configuration.MavenLaunchDelegate.getDynamicProperties(org.eclipse.debug.core.ILaunchConfiguration,java.util.List) makes inefficient use of keySet iterator instead of entrySet iterator | 200 |
| WMI: Method org.mevenide.ui.eclipse.launch.configuration.MavenLaunchDelegate.getOptions(org.eclipse.debug.core.ILaunchConfiguration,java.util.List) makes inefficient use of keySet iterator instead of entrySet iterator | 257 |
| REC: Method org.mevenide.ui.eclipse.launch.configuration.MavenLaunchDelegate.getValue(String) catches Exception, but Exception is not thrown in the try block and RuntimeException is not explicitly caught | 366 |
| CD: Class org.mevenide.ui.eclipse.launch.configuration.MavenLaunchDelegate has a circular dependency with other classes. | 0 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.launch.configuration.MavenLaunchShortcut.launch(org.eclipse.core.runtime.IPath) | 138 |
| Violation | Line |
|---|---|
| DB: Method org.mevenide.ui.eclipse.pom.manager.DefaultPOMManager.visit(org.eclipse.core.resources.IResourceDelta) uses the same code for two branches | 142 |
| DB: Method org.mevenide.ui.eclipse.pom.manager.DefaultPOMManager.visit(org.eclipse.core.resources.IResourceDelta) uses the same code for two branches | 143 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.pom.manager.DefaultPOMManager.fileToProjectMap | 0 |
| Violation | Line |
|---|---|
| REC: Method org.mevenide.ui.eclipse.pom.validation.ValidationJob.run(org.eclipse.core.runtime.IProgressMonitor) catches Exception, but Exception is not thrown in the try block and RuntimeException is not explicitly caught | 83 |
| Violation | Line |
|---|---|
| WMI: Method org.mevenide.ui.eclipse.preferences.dynamic.DynamicPreferencesManager.getPreferences() makes inefficient use of keySet iterator instead of entrySet iterator | 57 |
| WMI: Method org.mevenide.ui.eclipse.preferences.dynamic.DynamicPreferencesManager.importProperties(java.util.Properties) makes inefficient use of keySet iterator instead of entrySet iterator | 80 |
| WMI: Method org.mevenide.ui.eclipse.preferences.dynamic.DynamicPreferencesManager.exportProperties() makes inefficient use of keySet iterator instead of entrySet iterator | 99 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.preferences.dynamic.PluginProperty defines equals and uses Object.hashCode() | 0 |
| Violation | Line |
|---|---|
| REC: Method org.mevenide.ui.eclipse.preferences.pages.PluginsRootPreferencePage$1.widgetSelected(org.eclipse.swt.events.SelectionEvent) catches Exception, but Exception is not thrown in the try block and RuntimeException is not explicitly caught | 120 |
| Violation | Line |
|---|---|
| SBSC: Method org.mevenide.ui.eclipse.repository.view.DownloadJob.run(org.eclipse.core.runtime.IProgressMonitor) concatenates strings using + in a loop | 81 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.repository.view.RepositoryContentProvider.getChildren(Object) return a zero length array rather than null? | 56 |
| PZLA: Should org.mevenide.ui.eclipse.repository.view.RepositoryContentProvider.getChildren(Object) return a zero length array rather than null? | 73 |
| PZLA: Should org.mevenide.ui.eclipse.repository.view.RepositoryContentProvider.getElements(Object) return a zero length array rather than null? | 100 |
| Violation | Line |
|---|---|
| SBSC: Method org.mevenide.ui.eclipse.repository.view.RepositoryList.saveUserDefinedRepositories(java.util.List) concatenates strings using + in a loop | 107 |
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.repository.view.RepositoryList.getRepositoryList(String,java.util.List) | 89 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.repository.view.SearchResultView$1.getElements(Object) return a zero length array rather than null? | 92 |
| SIC: The class org.mevenide.ui.eclipse.repository.view.SearchResultView$1 could be refactored into a named _static_ inner class | 0 |
| Violation | Line |
|---|---|
| Dm: org.mevenide.ui.eclipse.sync.action.ToggleViewAction.directionChanged(int) invokes dubious Boolean constructor; use Boolean.valueOf(...) instead | 47 |
| Violation | Line |
|---|---|
| OS: org.mevenide.ui.eclipse.sync.model.ArtifactNode.addIgnoreLine(String,org.eclipse.core.resources.IFile) may fail to close stream on exception | 91 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.sync.model.Directory defines equals and uses Object.hashCode() | 0 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.sync.model.Directory.type | 0 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.sync.model.DirectoryNode defines equals and uses Object.hashCode() | 0 |
| EI: org.mevenide.ui.eclipse.sync.model.DirectoryNode.getChildren() may expose internal representation by returning org.mevenide.ui.eclipse.sync.model.DirectoryNode.excludeNodes | 132 |
| EI2: org.mevenide.ui.eclipse.sync.model.DirectoryNode.setExcludeNodes(ExcludeNode[]) may expose internal representation by storing an externally mutable object into org.mevenide.ui.eclipse.sync.model.DirectoryNode.excludeNodes | 264 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.sync.model.EclipseProjectNode defines equals and uses Object.hashCode() | 0 |
| EI: org.mevenide.ui.eclipse.sync.model.EclipseProjectNode.getChildren() may expose internal representation by returning org.mevenide.ui.eclipse.sync.model.EclipseProjectNode.mavenProjectNodes | 185 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.sync.model.ExcludeNode.getChildren() return a zero length array rather than null? | 49 |
| CD: Class org.mevenide.ui.eclipse.sync.model.ExcludeNode has a circular dependency with other classes. | 0 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.sync.model.MavenArtifactNode defines equals and uses Object.hashCode() | 0 |
| EI: org.mevenide.ui.eclipse.sync.model.MavenArtifactNode.getChildren() may expose internal representation by returning org.mevenide.ui.eclipse.sync.model.MavenArtifactNode.properties | 95 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.sync.model.MavenProjectNode defines equals and uses Object.hashCode() | 0 |
| PZLA: Should org.mevenide.ui.eclipse.sync.model.MavenProjectNode.getEclipseExclusionFilterNodes(String,DirectoryNode) return a zero length array rather than null? | 344 |
| RCN: Nullcheck of value previously dereferenced org.mevenide.ui.eclipse.sync.model.MavenProjectNode.initializeDirectories(org.apache.maven.project.Project) | 280 |
| SA: Self assignment of local variable in org.mevenide.ui.eclipse.sync.model.MavenProjectNode.createSourceFolderNodes(java.util.List) | 325 |
| ITA: Method org.mevenide.ui.eclipse.sync.model.MavenProjectNode.initializeArtifacts(org.apache.maven.project.Project) uses Collection.toArray() with zero-length array argument | 218 |
| ITA: Method org.mevenide.ui.eclipse.sync.model.MavenProjectNode.joinEclipseProjectArtifacts() uses Collection.toArray() with zero-length array argument | 252 |
| ITA: Method org.mevenide.ui.eclipse.sync.model.MavenProjectNode.joinEclipseSourceFolders() uses Collection.toArray() with zero-length array argument | 318 |
| ITA: Method org.mevenide.ui.eclipse.sync.model.MavenProjectNode.createDirectoryNodes(java.util.Map) uses Collection.toArray() with zero-length array argument | 363 |
| WMI: Method org.mevenide.ui.eclipse.sync.model.MavenProjectNode.reverseMap(java.util.Map) makes inefficient use of keySet iterator instead of entrySet iterator | 296 |
| WMI: Method org.mevenide.ui.eclipse.sync.model.MavenProjectNode.createDirectoryNodes(java.util.Map) makes inefficient use of keySet iterator instead of entrySet iterator | 354 |
| REC: Method org.mevenide.ui.eclipse.sync.model.MavenProjectNode.initializeOutputFolders() catches Exception, but Exception is not thrown in the try block and RuntimeException is not explicitly caught | 125 |
| CD: Class org.mevenide.ui.eclipse.sync.model.MavenProjectNode has a circular dependency with other classes. | 0 |
| CD: Class org.mevenide.ui.eclipse.sync.model.MavenProjectNode has a circular dependency with other classes. | 0 |
| CD: Class org.mevenide.ui.eclipse.sync.model.MavenProjectNode has a circular dependency with other classes. | 0 |
| Violation | Line |
|---|---|
| HE: org.mevenide.ui.eclipse.sync.model.PropertyNode defines equals and uses Object.hashCode() | 0 |
| PZLA: Should org.mevenide.ui.eclipse.sync.model.PropertyNode.getChildren() return a zero length array rather than null? | 62 |
| CD: Class org.mevenide.ui.eclipse.sync.model.PropertyNode has a circular dependency with other classes. | 0 |
| Violation | Line |
|---|---|
| DLS: Dead store to local variable in method org.mevenide.ui.eclipse.sync.view.PomChoiceDialog.getProjectLabel(Object) | 201 |
| UwF: Field not initialized in constructor: org.mevenide.ui.eclipse.sync.view.PomChoiceDialog.okButton | 0 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.sync.view.SynchronizationNodeProvider.getChildren(Object) return a zero length array rather than null? | 41 |
| Violation | Line |
|---|---|
| PZLA: Should org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.computeContextInformation(org.eclipse.jface.text.ITextViewer,int) return a zero length array rather than null? | 471 |
| PZLA: Should org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.getContextInformationAutoActivationCharacters() return a zero length array rather than null? | 478 |
| WMI: Method org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.computeTags(org.eclipse.jface.text.IDocument,org.mevenide.ui.eclipse.editors.jelly.XMLNode,int) makes inefficient use of keySet iterator instead of entrySet iterator | 131 |
| WMI: Method org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.computeAttributes(org.eclipse.jface.text.IDocument,org.mevenide.ui.eclipse.editors.jelly.XMLNode,int) makes inefficient use of keySet iterator instead of entrySet iterator | 354 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.getNodeAt(org.eclipse.jface.text.IDocument,int) | 93 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.findLastOpenTag(org.eclipse.jface.text.IDocument,org.mevenide.ui.eclipse.editors.jelly.XMLNode) | 282 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.findLastOpenTag(org.eclipse.jface.text.IDocument,org.mevenide.ui.eclipse.editors.jelly.XMLNode) | 298 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.findLastOpenTag(org.eclipse.jface.text.IDocument,org.mevenide.ui.eclipse.editors.jelly.XMLNode) | 286 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.findLastOpenTag(org.eclipse.jface.text.IDocument,org.mevenide.ui.eclipse.editors.jelly.XMLNode) | 283 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.findLastOpenTag(org.eclipse.jface.text.IDocument,org.mevenide.ui.eclipse.editors.jelly.XMLNode) | 302 |
| BC: Unchecked/unconfirmed cast from org.eclipse.jface.text.Position to class org.mevenide.ui.eclipse.editors.jelly.XMLNode in org.mevenide.ui.eclipse.editors.jelly.contentassist.JellyContentAssistProcessor.findLastOpenTag(org.eclipse.jface.text.IDocument,org.mevenide.ui.eclipse.editors.jelly.XMLNode) | 299 |