Found reliance on default encoding in org.aavso.tools.vstar.auth.AAVSOPostUserPassXMLAuthenticationSource.authenticate(String, String): String.getBytes()

Found reliance on default encoding in org.aavso.tools.vstar.auth.Auth0JSONAutheticationSource.authenticate(String, String): new java.io.InputStreamReader(InputStream)

new org.aavso.tools.vstar.data.CommentCodes(String) invokes inefficient new String() constructor

org.aavso.tools.vstar.data.CommentCodes.getCommentcodes() may expose internal representation by returning CommentCodes.commentcodes

The class name org.aavso.tools.vstar.data.Property$propType doesn't start with an upper case letter

Public static org.aavso.tools.vstar.data.SeriesType.getSeriesColorChangeNotifier() may expose internal representation by returning SeriesType.seriesColorChangeNotifier

Public static org.aavso.tools.vstar.data.SeriesType.getSeriesSizeChangeNotifier() may expose internal representation by returning SeriesType.seriesSizeChangeNotifier

Public static org.aavso.tools.vstar.data.SeriesType.values() may expose internal representation by returning SeriesType.values

org.aavso.tools.vstar.data.SeriesType.updateSeriesColorMap(Map) makes inefficient use of keySet iterator instead of entrySet iterator

org.aavso.tools.vstar.data.SeriesType.updateSeriesSizeMap(Map) makes inefficient use of keySet iterator instead of entrySet iterator

org.aavso.tools.vstar.data.ValidObservation.getDetails() may expose internal representation by returning ValidObservation.details

org.aavso.tools.vstar.data.ValidObservation.getMagnitude() may expose internal representation by returning ValidObservation.magnitude

org.aavso.tools.vstar.data.ValidObservation.setMagnitude(Magnitude) may expose internal representation by storing an externally mutable object into ValidObservation.magnitude

Public static org.aavso.tools.vstar.data.ValidObservation.getDetailTitles() may expose internal representation by returning ValidObservation.detailTitles

Public static org.aavso.tools.vstar.data.ValidObservation.getDetailTypes() may expose internal representation by returning ValidObservation.detailTypes

Public static org.aavso.tools.vstar.data.ValidObservation.getStandardDetailKeys() may expose internal representation by returning ValidObservation.standardDetailKeys

Write to static field org.aavso.tools.vstar.data.ValidObservation.detailIndex from instance method org.aavso.tools.vstar.data.ValidObservation.addDetail(String, Property, String)

org.aavso.tools.vstar.data.ValidObservation.toString() makes inefficient use of keySet iterator instead of entrySet iterator

Exception thrown in class org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher at new org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher(Object, ObservationMatcherOp, ObservationMatcherOp[]) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Exception thrown in class org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher at new org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher(ObservationMatcherOp[]) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher.getMatcherOps() may expose internal representation by returning AbstractObservationFieldMatcher.ops

org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher.getObservationSelectionMessage() may expose internal representation by returning AbstractObservationFieldMatcher.observationSelectionMessage

new org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher(Object, ObservationMatcherOp, ObservationMatcherOp[]) may expose internal representation by storing an externally mutable object into AbstractObservationFieldMatcher.ops

org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher.setSelectedObservationMessage(ObservationSelectionMessage) may expose internal representation by storing an externally mutable object into AbstractObservationFieldMatcher.observationSelectionMessage

Switch statement found in org.aavso.tools.vstar.data.filter.BooleanFieldMatcher.matches(ValidObservation) where default case is missing

org.aavso.tools.vstar.data.filter.DoubleFieldMatcher.ops should be package protected

Switch statement found in org.aavso.tools.vstar.data.filter.DoubleFieldMatcher.matches(ValidObservation) where default case is missing

org.aavso.tools.vstar.data.filter.ObservationFilter.getMatchers() may expose internal representation by returning ObservationFilter.matchers

org.aavso.tools.vstar.data.filter.ObservationFilter.MATCHERS is a mutable collection

org.aavso.tools.vstar.data.filter.ObservationMatcherOp.toString() may return null

org.aavso.tools.vstar.data.filter.PhaseFieldMatcher.setSelectedObservationMessage(ObservationSelectionMessage) may expose internal representation by storing an externally mutable object into PhaseFieldMatcher.observationSelectionMessage

Switch statement found in org.aavso.tools.vstar.data.filter.SeriesTypeFieldMatcher.matches(ValidObservation) where default case is missing

Switch statement found in org.aavso.tools.vstar.data.filter.StringFieldMatcher.matches(ValidObservation) where default case is missing

org.aavso.tools.vstar.data.validation.AAVSOFormatFieldInfoSource.getFieldIndexMap() may expose internal representation by returning AAVSOFormatFieldInfoSource.fieldIndexMap

Exception thrown in class org.aavso.tools.vstar.data.validation.CKMagValidator at new org.aavso.tools.vstar.data.validation.CKMagValidator(String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

new org.aavso.tools.vstar.data.validation.CommonTextFormatValidator(String, CsvReader, int, int, String, IFieldInfoSource) may expose internal representation by storing an externally mutable object into CommonTextFormatValidator.lineReader

Unread field: org.aavso.tools.vstar.data.validation.JulianDayValidator.pattern; should this field be static?

Unread field: org.aavso.tools.vstar.data.validation.MagnitudeFieldValidator.MAG_INDEX; should this field be static?

Unread field: org.aavso.tools.vstar.data.validation.MagnitudeFieldValidator.MAG_MODIFIER_INDEX; should this field be static?

Unread field: org.aavso.tools.vstar.data.validation.MagnitudeFieldValidator.UNCERTAINTY_INDEX; should this field be static?

org.aavso.tools.vstar.data.validation.SimpleFormatFieldInfoSource.getFieldIndexMap() may expose internal representation by returning SimpleFormatFieldInfoSource.fieldIndexMap

org.aavso.tools.vstar.exception.ObservationValidationWarning.getObservation() may expose internal representation by returning ObservationValidationWarning.ob

new org.aavso.tools.vstar.exception.ObservationValidationWarning(String, ValidObservation) may expose internal representation by storing an externally mutable object into ObservationValidationWarning.ob

new org.aavso.tools.vstar.exception.ObservationValidationWarning(ValidObservation) may expose internal representation by storing an externally mutable object into ObservationValidationWarning.ob

Found reliance on default encoding in org.aavso.tools.vstar.input.AbstractObservationRetriever.removeNegativeBytes(String): new String(byte[], int, int)

Found reliance on default encoding in org.aavso.tools.vstar.input.AbstractObservationRetriever.removeNegativeBytes(String): String.getBytes()

org.aavso.tools.vstar.input.AbstractObservationRetriever.getInvalidObservations() may expose internal representation by returning AbstractObservationRetriever.invalidObservations

org.aavso.tools.vstar.input.AbstractObservationRetriever.getValidObservationCategoryMap() may expose internal representation by returning AbstractObservationRetriever.validObservationCategoryMap

org.aavso.tools.vstar.input.AbstractObservationRetriever.getValidObservations() may expose internal representation by returning AbstractObservationRetriever.validObservations

new org.aavso.tools.vstar.input.database.AuthenticationTask(List, String, String) may expose internal representation by storing an externally mutable object into AuthenticationTask.authenticators

org.aavso.tools.vstar.input.database.Authenticator.authenticate() invokes inefficient new String(String) constructor

new org.aavso.tools.vstar.input.text.ObservationFieldSplitter(CsvReader, int, int) may expose internal representation by storing an externally mutable object into ObservationFieldSplitter.lineReader

new org.aavso.tools.vstar.input.text.ObservationSourceAnalyser(LineNumberReader, String) may expose internal representation by storing an externally mutable object into ObservationSourceAnalyser.obsSource

new org.aavso.tools.vstar.input.text.TextFormatObservationReader(LineNumberReader, ObservationSourceAnalyser, String) may expose internal representation by storing an externally mutable object into TextFormatObservationReader.reader

org.aavso.tools.vstar.plugin.CustomFilterPluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.GeneralToolPluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.ModelCreatorPluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.ObservationSinkPluginBase.test() has Boolean return type and returns explicit null

Found reliance on default encoding in org.aavso.tools.vstar.plugin.ObservationSourcePluginBase.getTestRetriever(String[], String): String.getBytes()

org.aavso.tools.vstar.plugin.ObservationSourcePluginBase.getInputStreams() may expose internal representation by returning ObservationSourcePluginBase.inputStreams

org.aavso.tools.vstar.plugin.ObservationSourcePluginBase.setInputInfo(List, String) may expose internal representation by storing an externally mutable object into ObservationSourcePluginBase.inputStreams

org.aavso.tools.vstar.plugin.ObservationSourcePluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.ObservationToolPluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.ObservationTransformerPluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.ob.sink.impl.CommonTextFormatSinkPluginBase.getDelimiterNameValuePairs() may expose internal representation by returning CommonTextFormatSinkPluginBase.DELIMS

org.aavso.tools.vstar.plugin.ob.sink.impl.CommonTextFormatSinkPluginBase.getDelimiterSuffixValuePairs() may expose internal representation by returning CommonTextFormatSinkPluginBase.SUFFIXES

org.aavso.tools.vstar.plugin.ob.sink.impl.CommonTextFormatSinkPluginBase.DELIMS is a mutable collection which should be package protected

org.aavso.tools.vstar.plugin.ob.sink.impl.CommonTextFormatSinkPluginBase.SUFFIXES is a mutable collection which should be package protected

Call to java.util.Locale.equals(String) in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase.getDescription()

Call to java.util.Locale.equals(String) in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase.getDisplayName()

org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase.getInputName() concatenates strings using + in a loop

org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase.getURLs() makes inefficient use of keySet iterator instead of entrySet iterator

Found reliance on default encoding in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase$AAVSOPhotometryURLObservationRetriever.retrieveAAVSOPhotometryURLObs(InputStream, SeriesType): new java.io.InputStreamReader(InputStream)

Call to java.util.Locale.equals(String) in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase$AAVSOPhotometryURLObservationRetriever.getSourceType()

Switch statement found in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase$AAVSOPhotometryURLSearchParameterDialog$2.actionPerformed(ActionEvent) where default case is missing

Redundant nullcheck of id, which is known to be non-null in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceCSVObservationSourcePlugin$VSXCSVObservationRetriever.retrieveNextObservation(CsvReader)

Exception thrown in class org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase at new org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase(String, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Dead store to starName in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.getURLs()

org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.setInfo(StarInfo) may expose internal representation by storing an externally mutable object into AIDWebServiceObservationSourcePluginBase.info

Comparison of String parameter using == or != in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.getMType(String)

Comparison of String parameter using == or != in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.getMType(String)

Load of known null value in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.getMType(String)

Redundant nullcheck of id, which is known to be non-null in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceXMLAttributeObservationSourcePlugin$VSXAIDAttributeObservationRetriever.retrieveObservation(INodeSequence)

new org.aavso.tools.vstar.plugin.ob.src.impl.NamedNodeMapSequence(NamedNodeMap) may expose internal representation by storing an externally mutable object into NamedNodeMapSequence.nodeMap

Found reliance on default encoding in org.aavso.tools.vstar.plugin.ob.src.impl.TextFormatObservationSourcePlugin.getObservationRetriever(): new java.io.InputStreamReader(InputStream)

Found reliance on default encoding in org.aavso.tools.vstar.plugin.ob.src.impl.TextFormatObservationSourcePlugin.getObservationRetriever(): String.getBytes()

org.aavso.tools.vstar.plugin.ob.src.impl.TextFormatObservationSourcePlugin.getObservationRetriever() may fail to close stream

org.aavso.tools.vstar.plugin.period.PeriodAnalysisPluginBase.test() has Boolean return type and returns explicit null

Found reliance on default encoding in org.aavso.tools.vstar.scripting.ScriptRunner.runScript(File): new java.io.FileReader(File)

Public static org.aavso.tools.vstar.scripting.ScriptRunner.getInstance() may expose internal representation by returning ScriptRunner.instance

Class (org.aavso.tools.vstar.scripting.ScriptRunner) using singleton design pattern has non-private constructor.

Incorrect lazy initialization of static field org.aavso.tools.vstar.scripting.VStarScriptingAPI.instance in org.aavso.tools.vstar.scripting.VStarScriptingAPI.getInstance()

Public static org.aavso.tools.vstar.scripting.VStarScriptingAPI.getInstance() may expose internal representation by returning VStarScriptingAPI.instance

Possible null pointer dereference of dcdft in org.aavso.tools.vstar.scripting.VStarScriptingAPI.dcdftCommon(String, DcDftAnalysisType, double, double, double) on exception path

org.aavso.tools.vstar.scripting.VStarScriptingAPI.getSeries() concatenates strings using + in a loop

org.aavso.tools.vstar.scripting.VStarScriptingAPI.pause(long) calls Thread.sleep() with a lock held

org.aavso.tools.vstar.ui.MainFrame.getStatusPane() may expose internal representation by returning MainFrame.statusPane

Dead store to lastGroup in org.aavso.tools.vstar.ui.MenuBar.createAnalysisMenu()

new org.aavso.tools.vstar.ui.MenuBar(IMainUI, UIType) may expose internal representation by storing an externally mutable object into MenuBar.parent

org.aavso.tools.vstar.ui.MenuBar.minimalUIExclusions is a mutable collection which should be package protected

org.aavso.tools.vstar.ui.NamedComponent.getComponent() may expose internal representation by returning NamedComponent.component

new org.aavso.tools.vstar.ui.NamedComponent(String, Component) may expose internal representation by storing an externally mutable object into NamedComponent.component

new org.aavso.tools.vstar.ui.NamedComponent(String, Component, String) may expose internal representation by storing an externally mutable object into NamedComponent.component

Exception thrown in class org.aavso.tools.vstar.ui.NumberSelectionPane at new org.aavso.tools.vstar.ui.NumberSelectionPane(String, double, double, double, double, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.ToolBar.retrieveToolBarIcons() invokes System.exit(...), which shuts down the entire virtual machine

new org.aavso.tools.vstar.ui.ToolBar(MenuBar) may expose internal representation by storing an externally mutable object into ToolBar.menuBar

Redundant nullcheck of org.aavso.tools.vstar.ui.ToolBar.newStarFromDatabaseIcon, which is known to be non-null in org.aavso.tools.vstar.ui.ToolBar.retrieveToolBarIcons()

Repeated conditional test in org.aavso.tools.vstar.ui.ToolBar.retrieveToolBarIcons()

org.aavso.tools.vstar.ui.VStar.LOGGER isn't final but should be

Exception is caught when Exception is not thrown in org.aavso.tools.vstar.ui.VStar.<static initializer for VStar>()

Exceptional return value of java.io.File.mkdir() ignored in org.aavso.tools.vstar.ui.VStar.<static initializer for VStar>()

org.aavso.tools.vstar.ui.VStar$4.run() invokes System.exit(...), which shuts down the entire virtual machine

Shared primitive variable "obsTextProvided" in one thread may not yield the value of the most recent write from another thread

org.aavso.tools.vstar.ui.dialog.AdditiveLoadFileOrUrlChooser.getDefaultFileExtensions() may expose internal representation by returning AdditiveLoadFileOrUrlChooser.extensions

Switch statement found in org.aavso.tools.vstar.ui.dialog.AdditiveLoadFileOrUrlChooser.createPluginsList() where default case is missing

org.aavso.tools.vstar.ui.dialog.Checkbox.getUIComponent() may expose internal representation by returning Checkbox.checkbox

Exception thrown in class org.aavso.tools.vstar.ui.dialog.DateToJdDialog at new org.aavso.tools.vstar.ui.dialog.DateToJdDialog(String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Should org.aavso.tools.vstar.ui.dialog.DateToJdDialog$DoubleFieldSeconds be a _static_ inner class?

Nullcheck of delimiter2suffixes at line 100 of value previously dereferenced in org.aavso.tools.vstar.ui.dialog.DelimitedFieldFileSaveChooser.getSelectedFile()

new org.aavso.tools.vstar.ui.dialog.FileExtensionFilter(List) may expose internal representation by storing an externally mutable object into FileExtensionFilter.extensions

Dead store to statsTextArea in org.aavso.tools.vstar.ui.dialog.InfoDialog.createInfoPanel(List)

org.aavso.tools.vstar.ui.dialog.InfoDialog.createInfoPanel(List) makes inefficient use of keySet iterator instead of entrySet iterator

Found reliance on default encoding in org.aavso.tools.vstar.auth.AAVSOPostUserPassXMLAuthenticationSource.authenticate(String, String): String.getBytes()

Exception thrown in class org.aavso.tools.vstar.ui.dialog.LogDialog at new org.aavso.tools.vstar.ui.dialog.LogDialog() will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Found reliance on default encoding in org.aavso.tools.vstar.auth.Auth0JSONAutheticationSource.authenticate(String, String): new java.io.InputStreamReader(InputStream)

Found reliance on default encoding in org.aavso.tools.vstar.ui.dialog.LogDialog.createLogPanel(): new java.io.FileReader(String)

new org.aavso.tools.vstar.data.CommentCodes(String) invokes inefficient new String() constructor

new org.aavso.tools.vstar.ui.dialog.MultiEntryComponentDialog(String, String, List, Optional) may expose internal representation by storing an externally mutable object into MultiEntryComponentDialog.fields

org.aavso.tools.vstar.data.CommentCodes.getCommentcodes() may expose internal representation by returning CommentCodes.commentcodes

org.aavso.tools.vstar.ui.dialog.MultiEntryComponentDialog.okAction() concatenates strings using + in a loop

The class name org.aavso.tools.vstar.data.Property$propType doesn't start with an upper case letter

org.aavso.tools.vstar.ui.dialog.NumberFieldBase.getUIComponent() may expose internal representation by returning NumberFieldBase.textField

Public static org.aavso.tools.vstar.data.SeriesType.getSeriesColorChangeNotifier() may expose internal representation by returning SeriesType.seriesColorChangeNotifier

new org.aavso.tools.vstar.ui.dialog.ObservationDetailsDialog(ValidObservation) may expose internal representation by storing an externally mutable object into ObservationDetailsDialog.ob

Public static org.aavso.tools.vstar.data.SeriesType.getSeriesSizeChangeNotifier() may expose internal representation by returning SeriesType.seriesSizeChangeNotifier

org.aavso.tools.vstar.util.stats.epoch.AlphaOmegaMeanJDEpochStrategy stored into non-transient field PhaseParameterDialog.epochStrategy

Public static org.aavso.tools.vstar.data.SeriesType.values() may expose internal representation by returning SeriesType.values

new org.aavso.tools.vstar.ui.dialog.PlotControlDialog(String, ObservationAndMeanPlotPane, TimeElementsInBinSettingPane, NamedComponent, AnalysisType) may expose internal representation by storing an externally mutable object into PlotControlDialog.plotPane

org.aavso.tools.vstar.data.SeriesType.updateSeriesColorMap(Map) makes inefficient use of keySet iterator instead of entrySet iterator

new org.aavso.tools.vstar.ui.dialog.PlotControlDialog(String, ObservationAndMeanPlotPane, TimeElementsInBinSettingPane, NamedComponent, AnalysisType) may expose internal representation by storing an externally mutable object into PlotControlDialog.timeElementsInBinSettingPane

org.aavso.tools.vstar.data.SeriesType.updateSeriesSizeMap(Map) makes inefficient use of keySet iterator instead of entrySet iterator

new org.aavso.tools.vstar.ui.dialog.RadioButtonDialog(String, Collection, String) may expose internal representation by storing an externally mutable object into RadioButtonDialog.options

org.aavso.tools.vstar.data.ValidObservation.getDetails() may expose internal representation by returning ValidObservation.details

org.aavso.tools.vstar.ui.dialog.SelectableTextField.getUIComponent() may expose internal representation by returning SelectableTextField.textChooser

org.aavso.tools.vstar.data.ValidObservation.getMagnitude() may expose internal representation by returning ValidObservation.magnitude

new org.aavso.tools.vstar.ui.dialog.SeriesTypeCreationDialog(List) may expose internal representation by storing an externally mutable object into SeriesTypeCreationDialog.obs

org.aavso.tools.vstar.data.ValidObservation.setMagnitude(Magnitude) may expose internal representation by storing an externally mutable object into ValidObservation.magnitude

org.aavso.tools.vstar.ui.dialog.StarGroupSelectionPane.getStarGroups() may expose internal representation by returning StarGroupSelectionPane.starGroups

Public static org.aavso.tools.vstar.data.ValidObservation.getDetailTitles() may expose internal representation by returning ValidObservation.detailTitles

Exception thrown in class org.aavso.tools.vstar.ui.dialog.StarSelectorDialog at new org.aavso.tools.vstar.ui.dialog.StarSelectorDialog() will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Public static org.aavso.tools.vstar.data.ValidObservation.getDetailTypes() may expose internal representation by returning ValidObservation.detailTypes

Public static org.aavso.tools.vstar.ui.dialog.StarSelectorDialog.getInstance() may expose internal representation by returning StarSelectorDialog.instance

Class (org.aavso.tools.vstar.ui.dialog.StarSelectorDialog) using singleton design pattern directly or indirectly implements Serializable interface.

Public static org.aavso.tools.vstar.data.ValidObservation.getStandardDetailKeys() may expose internal representation by returning ValidObservation.standardDetailKeys

org.aavso.tools.vstar.ui.dialog.TextArea.getUIComponent() may expose internal representation by returning TextArea.textArea

Write to static field org.aavso.tools.vstar.data.ValidObservation.detailIndex from instance method org.aavso.tools.vstar.data.ValidObservation.addDetail(String, Property, String)

Exception thrown in class org.aavso.tools.vstar.ui.dialog.TextAreaTabs at new org.aavso.tools.vstar.ui.dialog.TextAreaTabs(List, List, int, int, boolean, boolean, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.data.ValidObservation.toString() makes inefficient use of keySet iterator instead of entrySet iterator

org.aavso.tools.vstar.ui.dialog.TextAreaTabs.getUIComponent() may expose internal representation by returning TextAreaTabs.tabs

Exception thrown in class org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher at new org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher(Object, ObservationMatcherOp, ObservationMatcherOp[]) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.dialog.TextDialog.getTextFields() may expose internal representation by returning TextDialog.textFields

Exception thrown in class org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher at new org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher(ObservationMatcherOp[]) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.dialog.TextField.getUIComponent() may expose internal representation by returning TextField.textField

org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher.getMatcherOps() may expose internal representation by returning AbstractObservationFieldMatcher.ops

new org.aavso.tools.vstar.ui.dialog.model.HarmonicInfoDialog(HarmonicSearchResultMessage, Component) may expose internal representation by storing an externally mutable object into HarmonicInfoDialog.interfaceComponent

org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher.getObservationSelectionMessage() may expose internal representation by returning AbstractObservationFieldMatcher.observationSelectionMessage

new org.aavso.tools.vstar.ui.dialog.model.HarmonicInfoDialog(HarmonicSearchResultMessage, Component) may expose internal representation by storing an externally mutable object into HarmonicInfoDialog.msg

new org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher(Object, ObservationMatcherOp, ObservationMatcherOp[]) may expose internal representation by storing an externally mutable object into AbstractObservationFieldMatcher.ops

org.aavso.tools.vstar.ui.dialog.model.HarmonicInfoDialog$3.actionPerformed(ActionEvent) concatenates strings using + in a loop

org.aavso.tools.vstar.data.filter.AbstractObservationFieldMatcher.setSelectedObservationMessage(ObservationSelectionMessage) may expose internal representation by storing an externally mutable object into AbstractObservationFieldMatcher.observationSelectionMessage

org.aavso.tools.vstar.ui.dialog.model.HarmonicInputDialog.getHarmonics() may expose internal representation by returning HarmonicInputDialog.harmonicsPerSelectedPeriod

Switch statement found in org.aavso.tools.vstar.data.filter.BooleanFieldMatcher.matches(ValidObservation) where default case is missing

new org.aavso.tools.vstar.ui.dialog.model.HarmonicInputDialog(Component, List, Map, String) may expose internal representation by storing an externally mutable object into HarmonicInputDialog.freqToHarmonics

org.aavso.tools.vstar.data.filter.DoubleFieldMatcher.ops should be package protected

org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane.getChart() may expose internal representation by returning PeriodAnalysis2DChartPane.chart

Switch statement found in org.aavso.tools.vstar.data.filter.DoubleFieldMatcher.matches(ValidObservation) where default case is missing

org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane.getModel() may expose internal representation by returning PeriodAnalysis2DChartPane.model

org.aavso.tools.vstar.data.filter.ObservationFilter.getMatchers() may expose internal representation by returning ObservationFilter.matchers

org.aavso.tools.vstar.data.filter.ObservationFilter.MATCHERS is a mutable collection

org.aavso.tools.vstar.data.filter.ObservationMatcherOp.toString() may return null

org.aavso.tools.vstar.data.filter.PhaseFieldMatcher.setSelectedObservationMessage(ObservationSelectionMessage) may expose internal representation by storing an externally mutable object into PhaseFieldMatcher.observationSelectionMessage

Switch statement found in org.aavso.tools.vstar.data.filter.SeriesTypeFieldMatcher.matches(ValidObservation) where default case is missing

Switch statement found in org.aavso.tools.vstar.data.filter.StringFieldMatcher.matches(ValidObservation) where default case is missing

org.aavso.tools.vstar.data.validation.AAVSOFormatFieldInfoSource.getFieldIndexMap() may expose internal representation by returning AAVSOFormatFieldInfoSource.fieldIndexMap

Exception thrown in class org.aavso.tools.vstar.data.validation.CKMagValidator at new org.aavso.tools.vstar.data.validation.CKMagValidator(String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

new org.aavso.tools.vstar.data.validation.CommonTextFormatValidator(String, CsvReader, int, int, String, IFieldInfoSource) may expose internal representation by storing an externally mutable object into CommonTextFormatValidator.lineReader

Unread field: org.aavso.tools.vstar.data.validation.JulianDayValidator.pattern; should this field be static?

Unread field: org.aavso.tools.vstar.data.validation.MagnitudeFieldValidator.MAG_INDEX; should this field be static?

Unread field: org.aavso.tools.vstar.data.validation.MagnitudeFieldValidator.MAG_MODIFIER_INDEX; should this field be static?

Unread field: org.aavso.tools.vstar.data.validation.MagnitudeFieldValidator.UNCERTAINTY_INDEX; should this field be static?

org.aavso.tools.vstar.data.validation.SimpleFormatFieldInfoSource.getFieldIndexMap() may expose internal representation by returning SimpleFormatFieldInfoSource.fieldIndexMap

org.aavso.tools.vstar.exception.ObservationValidationWarning.getObservation() may expose internal representation by returning ObservationValidationWarning.ob

new org.aavso.tools.vstar.exception.ObservationValidationWarning(String, ValidObservation) may expose internal representation by storing an externally mutable object into ObservationValidationWarning.ob

new org.aavso.tools.vstar.exception.ObservationValidationWarning(ValidObservation) may expose internal representation by storing an externally mutable object into ObservationValidationWarning.ob

Found reliance on default encoding in org.aavso.tools.vstar.input.AbstractObservationRetriever.removeNegativeBytes(String): new String(byte[], int, int)

Found reliance on default encoding in org.aavso.tools.vstar.input.AbstractObservationRetriever.removeNegativeBytes(String): String.getBytes()

org.aavso.tools.vstar.input.AbstractObservationRetriever.getInvalidObservations() may expose internal representation by returning AbstractObservationRetriever.invalidObservations

org.aavso.tools.vstar.input.AbstractObservationRetriever.getValidObservationCategoryMap() may expose internal representation by returning AbstractObservationRetriever.validObservationCategoryMap

org.aavso.tools.vstar.input.AbstractObservationRetriever.getValidObservations() may expose internal representation by returning AbstractObservationRetriever.validObservations

new org.aavso.tools.vstar.input.database.AuthenticationTask(List, String, String) may expose internal representation by storing an externally mutable object into AuthenticationTask.authenticators

new org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane(JFreeChart, PeriodAnalysis2DPlotModel, boolean) may expose internal representation by storing an externally mutable object into PeriodAnalysis2DChartPane.chart

new org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane(JFreeChart, PeriodAnalysis2DPlotModel, boolean) may expose internal representation by storing an externally mutable object into PeriodAnalysis2DChartPane.model

Method org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane.datasetChanged(DatasetChangeEvent) seems to be useless

Doomed test for equality to NaN in org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane$3.update(PeriodAnalysisSelectionMessage)

Exception thrown in class org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DResultDialog at new org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DResultDialog(String, String, IPeriodAnalysisAlgorithm, PeriodAnalysisCoordinateType[], PeriodAnalysisCoordinateType) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysisDataTablePane.getTable() may expose internal representation by returning PeriodAnalysisDataTablePane.table

new org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysisDataTablePane(PeriodAnalysisDataTableModel, IPeriodAnalysisAlgorithm, boolean) may expose internal representation by storing an externally mutable object into PeriodAnalysisDataTablePane.model

org.aavso.tools.vstar.ui.dialog.period.refinement.PeriodGatheringPane.getPeriods() may expose internal representation by returning PeriodGatheringPane.periods

new org.aavso.tools.vstar.ui.dialog.period.wwz.WWZDataTablePane(WWZDataTableModel) may expose internal representation by storing an externally mutable object into WWZDataTablePane.model

org.aavso.tools.vstar.ui.dialog.period.wwz.WWZPlotPane.getChart() may expose internal representation by returning WWZPlotPane.chart

org.aavso.tools.vstar.ui.dialog.period.wwz.WWZPlotPane.getModel() may expose internal representation by returning WWZPlotPane.model

new org.aavso.tools.vstar.ui.dialog.period.wwz.WWZPlotPane(JFreeChart, WWZ2DPlotModel, double, double) may expose internal representation by storing an externally mutable object into WWZPlotPane.chart

new org.aavso.tools.vstar.ui.dialog.period.wwz.WWZPlotPane(JFreeChart, WWZ2DPlotModel, double, double) may expose internal representation by storing an externally mutable object into WWZPlotPane.model

new org.aavso.tools.vstar.ui.dialog.period.wwz.WeightedWaveletZTransformResultDialog(String, String, WeightedWaveletZTransform, WWZCoordinateType) may expose internal representation by storing an externally mutable object into WeightedWaveletZTransformResultDialog.wwt

new org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementDialog(PluginManager) may expose internal representation by storing an externally mutable object into PluginManagementDialog.manager

Unread field: org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementDialog.DIALOG_TITLE; should this field be static?

Unread field: org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementDialog.DIALOG_TITLE_MOD; should this field be static?

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementOperation.getManager() may expose internal representation by returning PluginManagementOperation.manager

new org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementOperation(PluginManager, String) may expose internal representation by storing an externally mutable object into PluginManagementOperation.manager

Found reliance on default encoding in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.retrieveRemotePluginInfo(String): new java.io.InputStreamReader(InputStream)

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getLibs() may expose internal representation by returning PluginManager.libs

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getLocalDescriptionsToJarName() may expose internal representation by returning PluginManager.localDescriptions

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getLocalPluginsByJarName() may expose internal representation by returning PluginManager.localPlugins

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getRemoteDescriptionsToJarName() may expose internal representation by returning PluginManager.remoteDescriptions

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getRemotePluginsByJarName() may expose internal representation by returning PluginManager.remotePlugins

Possible null pointer dereference in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.deleteAllPlugins() due to return value of called method

Possible null pointer dereference in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.deleteAllPlugins() due to return value of called method

Possible null pointer dereference in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.retrieveLocalPluginInfo() due to return value of called method

Possible null pointer dereference in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.retrieveLocalPluginInfo() due to return value of called method

Exceptional return value of java.io.File.mkdir() ignored in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.installPlugin(String, PluginManager$Operation)

Should org.aavso.tools.vstar.ui.dialog.prefs.ChartPropertiesSelectionPane$ChartFontLabel be a _static_ inner class?

Should org.aavso.tools.vstar.ui.dialog.prefs.ChartPropertiesSelectionPane$ColorRectComponent be a _static_ inner class?

Potentially ambiguous invocation of either an outer or inherited method org.aavso.tools.vstar.ui.dialog.AbstractOkCancelDialog.createButtonPane() in new org.aavso.tools.vstar.ui.dialog.prefs.ChartPropertiesSelectionPane$FontDialog(ChartPropertiesSelectionPane, Font)

Should org.aavso.tools.vstar.ui.dialog.prefs.ChartPropertiesSelectionPane$FontDialog be a _static_ inner class?

new org.aavso.tools.vstar.ui.dialog.prefs.LocaleSelectionPane() makes inefficient use of keySet iterator instead of entrySet iterator

org.aavso.tools.vstar.ui.dialog.prefs.NewGroupWithStarsDialog.getStarList() may expose internal representation by returning NewGroupWithStarsDialog.starList

Public static org.aavso.tools.vstar.ui.dialog.prefs.PreferencesDialog.getInstance() may expose internal representation by returning PreferencesDialog.instance

Class (org.aavso.tools.vstar.ui.dialog.prefs.PreferencesDialog) using singleton design pattern directly or indirectly implements Serializable interface.

org.aavso.tools.vstar.ui.dialog.prefs.SeriesColorSelectionPane.getChangedSeriesColorMap() may expose internal representation by returning SeriesColorSelectionPane.changedSeriesColorMap

org.aavso.tools.vstar.input.database.Authenticator.authenticate() invokes inefficient new String(String) constructor

org.aavso.tools.vstar.ui.dialog.prefs.SeriesSizeSelectionPane.getChangedSeriesSizeMap() may expose internal representation by returning SeriesSizeSelectionPane.changedSeriesSizeMap

new org.aavso.tools.vstar.input.text.ObservationFieldSplitter(CsvReader, int, int) may expose internal representation by storing an externally mutable object into ObservationFieldSplitter.lineReader

Should org.aavso.tools.vstar.ui.dialog.prefs.SeriesSizeSelectionPane$DotComponent be a _static_ inner class?

new org.aavso.tools.vstar.input.text.ObservationSourceAnalyser(LineNumberReader, String) may expose internal representation by storing an externally mutable object into ObservationSourceAnalyser.obsSource

org.aavso.tools.vstar.ui.dialog.series.AbstractSeriesVisibilityDialog.getNextPane() may expose internal representation by returning AbstractSeriesVisibilityDialog.nextPane

new org.aavso.tools.vstar.input.text.TextFormatObservationReader(LineNumberReader, ObservationSourceAnalyser, String) may expose internal representation by storing an externally mutable object into TextFormatObservationReader.reader

new org.aavso.tools.vstar.ui.dialog.series.AbstractSeriesVisibilityDialog(ISeriesInfoProvider, String, JPanel) may expose internal representation by storing an externally mutable object into AbstractSeriesVisibilityDialog.nextPane

org.aavso.tools.vstar.plugin.CustomFilterPluginBase.test() has Boolean return type and returns explicit null

Unwritten public or protected field: org.aavso.tools.vstar.ui.dialog.series.AbstractSeriesVisibilityDialog.seriesVisibilityPane

org.aavso.tools.vstar.plugin.GeneralToolPluginBase.test() has Boolean return type and returns explicit null

new org.aavso.tools.vstar.ui.dialog.series.MeanSourcePane(ObservationAndMeanPlotModel, ObservationAndMeanPlotPane) may expose internal representation by storing an externally mutable object into MeanSourcePane.obsPlotModel

org.aavso.tools.vstar.plugin.ModelCreatorPluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.ObservationSinkPluginBase.test() has Boolean return type and returns explicit null

new org.aavso.tools.vstar.ui.dialog.series.MeanSourcePane(ObservationAndMeanPlotModel, ObservationAndMeanPlotPane) may expose internal representation by storing an externally mutable object into MeanSourcePane.obsPlotPane

Found reliance on default encoding in org.aavso.tools.vstar.plugin.ObservationSourcePluginBase.getTestRetriever(String[], String): String.getBytes()

new org.aavso.tools.vstar.ui.dialog.series.MultipleSeriesSelectionDialog(SeriesVisibilityPane) may expose internal representation by storing an externally mutable object into MultipleSeriesSelectionDialog.seriesVisibilityPane

org.aavso.tools.vstar.plugin.ObservationSourcePluginBase.getInputStreams() may expose internal representation by returning ObservationSourcePluginBase.inputStreams

org.aavso.tools.vstar.ui.dialog.series.SeriesVisibilityPane.getVisibilityDeltaMap() may expose internal representation by returning SeriesVisibilityPane.visibilityDeltaMap

org.aavso.tools.vstar.plugin.ObservationSourcePluginBase.setInputInfo(List, String) may expose internal representation by storing an externally mutable object into ObservationSourcePluginBase.inputStreams

new org.aavso.tools.vstar.ui.dialog.series.SeriesVisibilityPane(ObservationAndMeanPlotModel, AnalysisType, boolean, boolean, boolean) may expose internal representation by storing an externally mutable object into SeriesVisibilityPane.obsPlotModel

org.aavso.tools.vstar.plugin.ObservationSourcePluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.ObservationToolPluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.ui.dialog.series.SeriesVisibilityPane.seriesVisibilityChange(Map) makes inefficient use of keySet iterator instead of entrySet iterator

org.aavso.tools.vstar.plugin.ObservationTransformerPluginBase.test() has Boolean return type and returns explicit null

org.aavso.tools.vstar.plugin.ob.sink.impl.CommonTextFormatSinkPluginBase.getDelimiterNameValuePairs() may expose internal representation by returning CommonTextFormatSinkPluginBase.DELIMS

org.aavso.tools.vstar.plugin.ob.sink.impl.CommonTextFormatSinkPluginBase.getDelimiterSuffixValuePairs() may expose internal representation by returning CommonTextFormatSinkPluginBase.SUFFIXES

org.aavso.tools.vstar.plugin.ob.sink.impl.CommonTextFormatSinkPluginBase.DELIMS is a mutable collection which should be package protected

org.aavso.tools.vstar.plugin.ob.sink.impl.CommonTextFormatSinkPluginBase.SUFFIXES is a mutable collection which should be package protected

Call to java.util.Locale.equals(String) in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase.getDescription()

Call to java.util.Locale.equals(String) in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase.getDisplayName()

org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase.getInputName() concatenates strings using + in a loop

org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase.getURLs() makes inefficient use of keySet iterator instead of entrySet iterator

Found reliance on default encoding in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase$AAVSOPhotometryURLObservationRetriever.retrieveAAVSOPhotometryURLObs(InputStream, SeriesType): new java.io.InputStreamReader(InputStream)

Call to java.util.Locale.equals(String) in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase$AAVSOPhotometryURLObservationRetriever.getSourceType()

Switch statement found in org.aavso.tools.vstar.plugin.ob.src.impl.AAVSOPhotometryURLObservationSourceBase$AAVSOPhotometryURLSearchParameterDialog$2.actionPerformed(ActionEvent) where default case is missing

Redundant nullcheck of id, which is known to be non-null in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceCSVObservationSourcePlugin$VSXCSVObservationRetriever.retrieveNextObservation(CsvReader)

Exception thrown in class org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase at new org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase(String, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Dead store to starName in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.getURLs()

org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.setInfo(StarInfo) may expose internal representation by storing an externally mutable object into AIDWebServiceObservationSourcePluginBase.info

Comparison of String parameter using == or != in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.getMType(String)

Comparison of String parameter using == or != in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.getMType(String)

Load of known null value in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceObservationSourcePluginBase.getMType(String)

Redundant nullcheck of id, which is known to be non-null in org.aavso.tools.vstar.plugin.ob.src.impl.AIDWebServiceXMLAttributeObservationSourcePlugin$VSXAIDAttributeObservationRetriever.retrieveObservation(INodeSequence)

new org.aavso.tools.vstar.plugin.ob.src.impl.NamedNodeMapSequence(NamedNodeMap) may expose internal representation by storing an externally mutable object into NamedNodeMapSequence.nodeMap

Found reliance on default encoding in org.aavso.tools.vstar.plugin.ob.src.impl.TextFormatObservationSourcePlugin.getObservationRetriever(): new java.io.InputStreamReader(InputStream)

Found reliance on default encoding in org.aavso.tools.vstar.plugin.ob.src.impl.TextFormatObservationSourcePlugin.getObservationRetriever(): String.getBytes()

org.aavso.tools.vstar.plugin.ob.src.impl.TextFormatObservationSourcePlugin.getObservationRetriever() may fail to close stream

org.aavso.tools.vstar.plugin.period.PeriodAnalysisPluginBase.test() has Boolean return type and returns explicit null

Found reliance on default encoding in org.aavso.tools.vstar.scripting.ScriptRunner.runScript(File): new java.io.FileReader(File)

Public static org.aavso.tools.vstar.scripting.ScriptRunner.getInstance() may expose internal representation by returning ScriptRunner.instance

Class (org.aavso.tools.vstar.scripting.ScriptRunner) using singleton design pattern has non-private constructor.

Incorrect lazy initialization of static field org.aavso.tools.vstar.scripting.VStarScriptingAPI.instance in org.aavso.tools.vstar.scripting.VStarScriptingAPI.getInstance()

Public static org.aavso.tools.vstar.scripting.VStarScriptingAPI.getInstance() may expose internal representation by returning VStarScriptingAPI.instance

Possible null pointer dereference of dcdft in org.aavso.tools.vstar.scripting.VStarScriptingAPI.dcdftCommon(String, DcDftAnalysisType, double, double, double) on exception path

org.aavso.tools.vstar.scripting.VStarScriptingAPI.getSeries() concatenates strings using + in a loop

org.aavso.tools.vstar.scripting.VStarScriptingAPI.pause(long) calls Thread.sleep() with a lock held

org.aavso.tools.vstar.ui.MainFrame.getStatusPane() may expose internal representation by returning MainFrame.statusPane

Dead store to lastGroup in org.aavso.tools.vstar.ui.MenuBar.createAnalysisMenu()

new org.aavso.tools.vstar.ui.MenuBar(IMainUI, UIType) may expose internal representation by storing an externally mutable object into MenuBar.parent

org.aavso.tools.vstar.ui.MenuBar.minimalUIExclusions is a mutable collection which should be package protected

new org.aavso.tools.vstar.ui.dialog.series.SingleSeriesSelectionDialog(ObservationAndMeanPlotModel) may expose internal representation by storing an externally mutable object into SingleSeriesSelectionDialog.obsPlotModel

new org.aavso.tools.vstar.ui.dialog.series.SingleSeriesSelectionPane(ObservationAndMeanPlotModel) may expose internal representation by storing an externally mutable object into SingleSeriesSelectionPane.obsPlotModel

org.aavso.tools.vstar.ui.mediator.DocumentManager.getStatsInfo() may expose internal representation by returning DocumentManager.statsInfo

Write to static field org.aavso.tools.vstar.ui.mediator.DocumentManager.filterNum from instance method org.aavso.tools.vstar.ui.mediator.DocumentManager.getNextUntitledFilterName()

org.aavso.tools.vstar.ui.mediator.Mediator.quit() invokes System.exit(...), which shuts down the entire virtual machine

org.aavso.tools.vstar.ui.mediator.Mediator.createNewStarObservationArtefacts(NewStarType, StarInfo, int, boolean) forces garbage collection; extremely dubious except in benchmarking code

org.aavso.tools.vstar.ui.mediator.Mediator.getAnalysisTypeChangeNotifier() may expose internal representation by returning Mediator.analysisTypeChangeNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getDiscrepantObservationNotifier() may expose internal representation by returning Mediator.discrepantObservationNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getDocumentManager() may expose internal representation by returning Mediator.documentManager

org.aavso.tools.vstar.ui.mediator.Mediator.getExcludedObservationNotifier() may expose internal representation by returning Mediator.excludedObservationNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getFilteredObservationNotifier() may expose internal representation by returning Mediator.filteredObservationNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getHarmonicSearchNotifier() may expose internal representation by returning Mediator.harmonicSearchNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getMeanSourceSeriesChangeNotifier() may expose internal representation by returning Mediator.meanSourceSeriesChangeNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getModelCreationNotifier() may expose internal representation by returning Mediator.modelCreationNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getModelDialog() may expose internal representation by returning Mediator.modelDialog

org.aavso.tools.vstar.ui.mediator.Mediator.getModelSelectionMessage() may expose internal representation by returning Mediator.modelSelectionMessage

org.aavso.tools.vstar.ui.mediator.Mediator.getModelSelectionNofitier() may expose internal representation by returning Mediator.modelSelectionNofitier

org.aavso.tools.vstar.ui.mediator.Mediator.getMultipleObservationSelectionNotifier() may expose internal representation by returning Mediator.multipleObservationSelectionNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getNewStarMessageList() may expose internal representation by returning Mediator.newStarMessageList

org.aavso.tools.vstar.ui.mediator.Mediator.getNewStarNotifier() may expose internal representation by returning Mediator.newStarNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getObsFilterDialog() may expose internal representation by returning Mediator.obsFilterDialog

org.aavso.tools.vstar.ui.mediator.Mediator.getObservationFiltersDialog() may expose internal representation by returning Mediator.observationFiltersDialog

org.aavso.tools.vstar.ui.mediator.Mediator.getObservationSelectionNotifier() may expose internal representation by returning Mediator.observationSelectionNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getPanRequestNotifier() may expose internal representation by returning Mediator.panRequestNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getPeriodAnalysisRefinementNotifier() may expose internal representation by returning Mediator.periodAnalysisRefinementNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getPeriodAnalysisSelectionNotifier() may expose internal representation by returning Mediator.periodAnalysisSelectionNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getPeriodChangeNotifier() may expose internal representation by returning Mediator.periodChangeNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getPhaseChangeNotifier() may expose internal representation by returning Mediator.phaseChangeNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getPhaseDialog() may expose internal representation by returning Mediator.phaseDialog

org.aavso.tools.vstar.ui.mediator.Mediator.getPhaseParameterDialog() may expose internal representation by returning Mediator.phaseParameterDialog

org.aavso.tools.vstar.ui.mediator.Mediator.getPhaseSelectionNotifier() may expose internal representation by returning Mediator.phaseSelectionNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getProgressNotifier() may expose internal representation by returning Mediator.progressNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getSeriesCreationNotifier() may expose internal representation by returning Mediator.seriesCreationNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getSeriesVisibilityChangeNotifier() may expose internal representation by returning Mediator.seriesVisibilityChangeNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getStopRequestNotifier() may expose internal representation by returning Mediator.stopRequestNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getUndoActionNotifier() may expose internal representation by returning Mediator.undoActionNotifier

org.aavso.tools.vstar.ui.mediator.Mediator.getUndoableActionManager() may expose internal representation by returning Mediator.undoableActionManager

org.aavso.tools.vstar.ui.mediator.Mediator.getValidObsList() may expose internal representation by returning Mediator.validObsList

org.aavso.tools.vstar.ui.mediator.Mediator.getValidObservationCategoryMap() may expose internal representation by returning Mediator.validObservationCategoryMap

org.aavso.tools.vstar.ui.mediator.Mediator.getVelaFileSaveDialog() may expose internal representation by returning Mediator.velaFileSaveDialog

org.aavso.tools.vstar.ui.mediator.Mediator.getVelaXMLsaveDialog() may expose internal representation by returning Mediator.velaXMLsaveDialog

org.aavso.tools.vstar.ui.mediator.Mediator.getZoomRequestNotifier() may expose internal representation by returning Mediator.zoomRequestNotifier

Public static org.aavso.tools.vstar.ui.mediator.Mediator.getInstance() may expose internal representation by returning Mediator.mediator

Public static org.aavso.tools.vstar.ui.mediator.Mediator.getUI() may expose internal representation by returning Mediator.ui

Possible null pointer dereference of obs in org.aavso.tools.vstar.ui.mediator.Mediator.saveSyntheticObsListToFile(Component, ObservationSinkPluginBase, ViewModeType, File, String)

Switch statement found in org.aavso.tools.vstar.ui.mediator.Mediator.saveSyntheticObsListToFile(Component, ObservationSinkPluginBase, ViewModeType, File, String) where one case falls through to the next case

Write to static field org.aavso.tools.vstar.ui.mediator.Mediator.ui from instance method org.aavso.tools.vstar.ui.mediator.Mediator.setUI(IMainUI)

Write to static field org.aavso.tools.vstar.ui.mediator.Mediator.ui from instance method new org.aavso.tools.vstar.ui.mediator.Mediator()

org.aavso.tools.vstar.ui.mediator.Mediator.createPhasePlotArtefacts(double, double, Map) makes inefficient use of keySet iterator instead of entrySet iterator

org.aavso.tools.vstar.ui.NamedComponent.getComponent() may expose internal representation by returning NamedComponent.component

org.aavso.tools.vstar.ui.mediator.StarInfo.getRetriever() may expose internal representation by returning StarInfo.retriever

new org.aavso.tools.vstar.ui.NamedComponent(String, Component) may expose internal representation by storing an externally mutable object into NamedComponent.component

new org.aavso.tools.vstar.ui.NamedComponent(String, Component, String) may expose internal representation by storing an externally mutable object into NamedComponent.component

Exception thrown in class org.aavso.tools.vstar.ui.NumberSelectionPane at new org.aavso.tools.vstar.ui.NumberSelectionPane(String, double, double, double, double, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.ToolBar.retrieveToolBarIcons() invokes System.exit(...), which shuts down the entire virtual machine

new org.aavso.tools.vstar.ui.ToolBar(MenuBar) may expose internal representation by storing an externally mutable object into ToolBar.menuBar

Redundant nullcheck of org.aavso.tools.vstar.ui.ToolBar.newStarFromDatabaseIcon, which is known to be non-null in org.aavso.tools.vstar.ui.ToolBar.retrieveToolBarIcons()

Repeated conditional test in org.aavso.tools.vstar.ui.ToolBar.retrieveToolBarIcons()

org.aavso.tools.vstar.ui.VStar.LOGGER isn't final but should be

Exception is caught when Exception is not thrown in org.aavso.tools.vstar.ui.VStar.<static initializer for VStar>()

Exceptional return value of java.io.File.mkdir() ignored in org.aavso.tools.vstar.ui.VStar.<static initializer for VStar>()

org.aavso.tools.vstar.ui.VStar$4.run() invokes System.exit(...), which shuts down the entire virtual machine

Shared primitive variable "obsTextProvided" in one thread may not yield the value of the most recent write from another thread

org.aavso.tools.vstar.ui.dialog.AdditiveLoadFileOrUrlChooser.getDefaultFileExtensions() may expose internal representation by returning AdditiveLoadFileOrUrlChooser.extensions

Switch statement found in org.aavso.tools.vstar.ui.dialog.AdditiveLoadFileOrUrlChooser.createPluginsList() where default case is missing

org.aavso.tools.vstar.ui.dialog.Checkbox.getUIComponent() may expose internal representation by returning Checkbox.checkbox

Exception thrown in class org.aavso.tools.vstar.ui.dialog.DateToJdDialog at new org.aavso.tools.vstar.ui.dialog.DateToJdDialog(String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Should org.aavso.tools.vstar.ui.dialog.DateToJdDialog$DoubleFieldSeconds be a _static_ inner class?

Nullcheck of delimiter2suffixes at line 100 of value previously dereferenced in org.aavso.tools.vstar.ui.dialog.DelimitedFieldFileSaveChooser.getSelectedFile()

new org.aavso.tools.vstar.ui.dialog.FileExtensionFilter(List) may expose internal representation by storing an externally mutable object into FileExtensionFilter.extensions

Dead store to statsTextArea in org.aavso.tools.vstar.ui.dialog.InfoDialog.createInfoPanel(List)

org.aavso.tools.vstar.ui.dialog.InfoDialog.createInfoPanel(List) makes inefficient use of keySet iterator instead of entrySet iterator

Exception thrown in class org.aavso.tools.vstar.ui.dialog.LogDialog at new org.aavso.tools.vstar.ui.dialog.LogDialog() will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Found reliance on default encoding in org.aavso.tools.vstar.ui.dialog.LogDialog.createLogPanel(): new java.io.FileReader(String)

new org.aavso.tools.vstar.ui.dialog.MultiEntryComponentDialog(String, String, List, Optional) may expose internal representation by storing an externally mutable object into MultiEntryComponentDialog.fields

org.aavso.tools.vstar.ui.dialog.MultiEntryComponentDialog.okAction() concatenates strings using + in a loop

org.aavso.tools.vstar.ui.dialog.NumberFieldBase.getUIComponent() may expose internal representation by returning NumberFieldBase.textField

new org.aavso.tools.vstar.ui.dialog.ObservationDetailsDialog(ValidObservation) may expose internal representation by storing an externally mutable object into ObservationDetailsDialog.ob

org.aavso.tools.vstar.util.stats.epoch.AlphaOmegaMeanJDEpochStrategy stored into non-transient field PhaseParameterDialog.epochStrategy

new org.aavso.tools.vstar.ui.dialog.PlotControlDialog(String, ObservationAndMeanPlotPane, TimeElementsInBinSettingPane, NamedComponent, AnalysisType) may expose internal representation by storing an externally mutable object into PlotControlDialog.plotPane

new org.aavso.tools.vstar.ui.dialog.PlotControlDialog(String, ObservationAndMeanPlotPane, TimeElementsInBinSettingPane, NamedComponent, AnalysisType) may expose internal representation by storing an externally mutable object into PlotControlDialog.timeElementsInBinSettingPane

new org.aavso.tools.vstar.ui.dialog.RadioButtonDialog(String, Collection, String) may expose internal representation by storing an externally mutable object into RadioButtonDialog.options

org.aavso.tools.vstar.ui.dialog.SelectableTextField.getUIComponent() may expose internal representation by returning SelectableTextField.textChooser

new org.aavso.tools.vstar.ui.dialog.SeriesTypeCreationDialog(List) may expose internal representation by storing an externally mutable object into SeriesTypeCreationDialog.obs

org.aavso.tools.vstar.ui.dialog.StarGroupSelectionPane.getStarGroups() may expose internal representation by returning StarGroupSelectionPane.starGroups

Exception thrown in class org.aavso.tools.vstar.ui.dialog.StarSelectorDialog at new org.aavso.tools.vstar.ui.dialog.StarSelectorDialog() will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

Public static org.aavso.tools.vstar.ui.dialog.StarSelectorDialog.getInstance() may expose internal representation by returning StarSelectorDialog.instance

Class (org.aavso.tools.vstar.ui.dialog.StarSelectorDialog) using singleton design pattern directly or indirectly implements Serializable interface.

org.aavso.tools.vstar.ui.dialog.TextArea.getUIComponent() may expose internal representation by returning TextArea.textArea

Exception thrown in class org.aavso.tools.vstar.ui.dialog.TextAreaTabs at new org.aavso.tools.vstar.ui.dialog.TextAreaTabs(List, List, int, int, boolean, boolean, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.dialog.TextAreaTabs.getUIComponent() may expose internal representation by returning TextAreaTabs.tabs

org.aavso.tools.vstar.ui.dialog.TextDialog.getTextFields() may expose internal representation by returning TextDialog.textFields

org.aavso.tools.vstar.ui.dialog.TextField.getUIComponent() may expose internal representation by returning TextField.textField

new org.aavso.tools.vstar.ui.dialog.model.HarmonicInfoDialog(HarmonicSearchResultMessage, Component) may expose internal representation by storing an externally mutable object into HarmonicInfoDialog.interfaceComponent

new org.aavso.tools.vstar.ui.dialog.model.HarmonicInfoDialog(HarmonicSearchResultMessage, Component) may expose internal representation by storing an externally mutable object into HarmonicInfoDialog.msg

org.aavso.tools.vstar.ui.dialog.model.HarmonicInfoDialog$3.actionPerformed(ActionEvent) concatenates strings using + in a loop

org.aavso.tools.vstar.ui.dialog.model.HarmonicInputDialog.getHarmonics() may expose internal representation by returning HarmonicInputDialog.harmonicsPerSelectedPeriod

new org.aavso.tools.vstar.ui.dialog.model.HarmonicInputDialog(Component, List, Map, String) may expose internal representation by storing an externally mutable object into HarmonicInputDialog.freqToHarmonics

org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane.getChart() may expose internal representation by returning PeriodAnalysis2DChartPane.chart

org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane.getModel() may expose internal representation by returning PeriodAnalysis2DChartPane.model

new org.aavso.tools.vstar.ui.mediator.StarInfo(AbstractObservationRetriever, String, String, Double, Double, String, String, String, RAInfo, DecInfo, Integer) may expose internal representation by storing an externally mutable object into StarInfo.retriever

org.aavso.tools.vstar.ui.mediator.StarInfo.setRetriever(AbstractObservationRetriever) may expose internal representation by storing an externally mutable object into StarInfo.retriever

org.aavso.tools.vstar.ui.mediator.message.AnalysisTypeChangeMessage.getMeansListPane() may expose internal representation by returning AnalysisTypeChangeMessage.meansListPane

org.aavso.tools.vstar.ui.mediator.message.AnalysisTypeChangeMessage.getObsAndMeanChartPane() may expose internal representation by returning AnalysisTypeChangeMessage.obsAndMeanChartPane

org.aavso.tools.vstar.ui.mediator.message.AnalysisTypeChangeMessage.getObsListPane() may expose internal representation by returning AnalysisTypeChangeMessage.obsListPane

new org.aavso.tools.vstar.ui.mediator.message.AnalysisTypeChangeMessage(AnalysisType, ObservationAndMeanPlotPane, ObservationListPane, SyntheticObservationListPane, ViewModeType) may expose internal representation by storing an externally mutable object into AnalysisTypeChangeMessage.meansListPane

new org.aavso.tools.vstar.ui.mediator.message.AnalysisTypeChangeMessage(AnalysisType, ObservationAndMeanPlotPane, ObservationListPane, SyntheticObservationListPane, ViewModeType) may expose internal representation by storing an externally mutable object into AnalysisTypeChangeMessage.obsAndMeanChartPane

new org.aavso.tools.vstar.ui.mediator.message.AnalysisTypeChangeMessage(AnalysisType, ObservationAndMeanPlotPane, ObservationListPane, SyntheticObservationListPane, ViewModeType) may expose internal representation by storing an externally mutable object into AnalysisTypeChangeMessage.obsListPane

org.aavso.tools.vstar.ui.mediator.message.DiscrepantObservationMessage.getObservation() may expose internal representation by returning DiscrepantObservationMessage.observation

new org.aavso.tools.vstar.ui.mediator.message.DiscrepantObservationMessage(ValidObservation, Object) may expose internal representation by storing an externally mutable object into DiscrepantObservationMessage.observation

org.aavso.tools.vstar.ui.mediator.message.ExcludedObservationMessage.getObservations() may expose internal representation by returning ExcludedObservationMessage.observations

new org.aavso.tools.vstar.ui.mediator.message.ExcludedObservationMessage(List, Object) may expose internal representation by storing an externally mutable object into ExcludedObservationMessage.observations

org.aavso.tools.vstar.ui.mediator.message.FilteredObservationMessage.getFilteredObs() may expose internal representation by returning FilteredObservationMessage.filteredObs

org.aavso.tools.vstar.ui.mediator.message.HarmonicSearchResultMessage.getHarmonics() may expose internal representation by returning HarmonicSearchResultMessage.harmonics

new org.aavso.tools.vstar.ui.mediator.message.HarmonicSearchResultMessage(Object, List, IPeriodAnalysisDatum, double) may expose internal representation by storing an externally mutable object into HarmonicSearchResultMessage.harmonics

org.aavso.tools.vstar.ui.mediator.message.MultipleObservationSelectionMessage.getObservations() may expose internal representation by returning MultipleObservationSelectionMessage.observations

new org.aavso.tools.vstar.ui.mediator.message.MultipleObservationSelectionMessage(List, Object) may expose internal representation by storing an externally mutable object into MultipleObservationSelectionMessage.observations

org.aavso.tools.vstar.ui.mediator.message.NewStarMessage.getLoadDate() may expose internal representation by returning NewStarMessage.loadDate

org.aavso.tools.vstar.ui.mediator.message.NewStarMessage.getObsCategoryMap() may expose internal representation by returning NewStarMessage.obsCategoryMap

org.aavso.tools.vstar.ui.mediator.message.NewStarMessage.getObservations() may expose internal representation by returning NewStarMessage.obs

org.aavso.tools.vstar.ui.mediator.message.NewStarMessage.getStarInfo() may expose internal representation by returning NewStarMessage.starInfo

new org.aavso.tools.vstar.ui.mediator.message.NewStarMessage(NewStarType, StarInfo, List, Map, double, double, String) may expose internal representation by storing an externally mutable object into NewStarMessage.obs

new org.aavso.tools.vstar.ui.mediator.message.NewStarMessage(NewStarType, StarInfo, List, Map, double, double, String) may expose internal representation by storing an externally mutable object into NewStarMessage.obsCategoryMap

new org.aavso.tools.vstar.ui.mediator.message.NewStarMessage(NewStarType, StarInfo, List, Map, double, double, String) may expose internal representation by storing an externally mutable object into NewStarMessage.starInfo

org.aavso.tools.vstar.ui.mediator.message.ObservationSelectionMessage.getObservation() may expose internal representation by returning ObservationSelectionMessage.observation

new org.aavso.tools.vstar.ui.mediator.message.ObservationSelectionMessage(ValidObservation, Object) may expose internal representation by storing an externally mutable object into ObservationSelectionMessage.observation

org.aavso.tools.vstar.ui.mediator.message.PeriodAnalysisRefinementMessage.getNewTopHits() may expose internal representation by returning PeriodAnalysisRefinementMessage.newTopHits

org.aavso.tools.vstar.ui.mediator.message.PeriodAnalysisRefinementMessage.getRefinedData() may expose internal representation by returning PeriodAnalysisRefinementMessage.refinedData

org.aavso.tools.vstar.ui.mediator.message.PeriodAnalysisRefinementMessage.getRefinedTopHits() may expose internal representation by returning PeriodAnalysisRefinementMessage.refinedTopHits

new org.aavso.tools.vstar.ui.mediator.message.PeriodAnalysisRefinementMessage(Object, Map, Map, List) may expose internal representation by storing an externally mutable object into PeriodAnalysisRefinementMessage.newTopHits

new org.aavso.tools.vstar.ui.mediator.message.PeriodAnalysisRefinementMessage(Object, Map, Map, List) may expose internal representation by storing an externally mutable object into PeriodAnalysisRefinementMessage.refinedData

new org.aavso.tools.vstar.ui.mediator.message.PeriodAnalysisRefinementMessage(Object, Map, Map, List) may expose internal representation by storing an externally mutable object into PeriodAnalysisRefinementMessage.refinedTopHits

org.aavso.tools.vstar.ui.mediator.message.PhaseChangeMessage.getSeriesVisibilityMap() may expose internal representation by returning PhaseChangeMessage.seriesVisibilityMap

new org.aavso.tools.vstar.ui.mediator.message.PhaseChangeMessage(Object, double, double, Map) may expose internal representation by storing an externally mutable object into PhaseChangeMessage.seriesVisibilityMap

org.aavso.tools.vstar.ui.mediator.message.PhaseChangeMessage.toString() makes inefficient use of keySet iterator instead of entrySet iterator

org.aavso.tools.vstar.ui.mediator.message.PhaseSelectionMessage.getSeriesVisibilityMap() may expose internal representation by returning PhaseSelectionMessage.seriesVisibilityMap

new org.aavso.tools.vstar.ui.mediator.message.PhaseSelectionMessage(Object, double, double, Map) may expose internal representation by storing an externally mutable object into PhaseSelectionMessage.seriesVisibilityMap

org.aavso.tools.vstar.ui.mediator.message.ProgressInfo.BUSY_PROGRESS isn't final but should be

org.aavso.tools.vstar.ui.mediator.message.ProgressInfo.CLEAR_PROGRESS isn't final but should be

org.aavso.tools.vstar.ui.mediator.message.ProgressInfo.COMPLETE_PROGRESS isn't final but should be

org.aavso.tools.vstar.ui.mediator.message.ProgressInfo.INCREMENT_PROGRESS isn't final but should be

org.aavso.tools.vstar.ui.mediator.message.ProgressInfo.START_PROGRESS isn't final but should be

org.aavso.tools.vstar.ui.mediator.message.SeriesCreationMessage.getObs() may expose internal representation by returning SeriesCreationMessage.obs

new org.aavso.tools.vstar.ui.mediator.message.SeriesCreationMessage(Object, SeriesType, List) may expose internal representation by storing an externally mutable object into SeriesCreationMessage.obs

org.aavso.tools.vstar.ui.mediator.message.SeriesVisibilityChangeMessage.getVisibleSeries() may expose internal representation by returning SeriesVisibilityChangeMessage.visibleSeries

new org.aavso.tools.vstar.ui.mediator.message.SeriesVisibilityChangeMessage(Object, Set) may expose internal representation by storing an externally mutable object into SeriesVisibilityChangeMessage.visibleSeries

org.aavso.tools.vstar.ui.mediator.message.UndoableActionType.toString() may return null

org.aavso.tools.vstar.ui.model.list.AAVSOFormatRawDataColumnInfoSource.COLUMN_NAMES is a mutable collection which should be package protected

Switch statement found in org.aavso.tools.vstar.ui.model.list.AAVSOFormatRawDataColumnInfoSource.getTableColumnClass(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.AAVSOFormatRawDataColumnInfoSource.getTableColumnTitle(int) where default case is missing

new org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane(JFreeChart, PeriodAnalysis2DPlotModel, boolean) may expose internal representation by storing an externally mutable object into PeriodAnalysis2DChartPane.chart

new org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane(JFreeChart, PeriodAnalysis2DPlotModel, boolean) may expose internal representation by storing an externally mutable object into PeriodAnalysis2DChartPane.model

Method org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane.datasetChanged(DatasetChangeEvent) seems to be useless

Doomed test for equality to NaN in org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DChartPane$3.update(PeriodAnalysisSelectionMessage)

Exception thrown in class org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DResultDialog at new org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysis2DResultDialog(String, String, IPeriodAnalysisAlgorithm, PeriodAnalysisCoordinateType[], PeriodAnalysisCoordinateType) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysisDataTablePane.getTable() may expose internal representation by returning PeriodAnalysisDataTablePane.table

new org.aavso.tools.vstar.ui.dialog.period.PeriodAnalysisDataTablePane(PeriodAnalysisDataTableModel, IPeriodAnalysisAlgorithm, boolean) may expose internal representation by storing an externally mutable object into PeriodAnalysisDataTablePane.model

org.aavso.tools.vstar.ui.dialog.period.refinement.PeriodGatheringPane.getPeriods() may expose internal representation by returning PeriodGatheringPane.periods

new org.aavso.tools.vstar.ui.dialog.period.wwz.WWZDataTablePane(WWZDataTableModel) may expose internal representation by storing an externally mutable object into WWZDataTablePane.model

org.aavso.tools.vstar.ui.dialog.period.wwz.WWZPlotPane.getChart() may expose internal representation by returning WWZPlotPane.chart

org.aavso.tools.vstar.ui.dialog.period.wwz.WWZPlotPane.getModel() may expose internal representation by returning WWZPlotPane.model

new org.aavso.tools.vstar.ui.dialog.period.wwz.WWZPlotPane(JFreeChart, WWZ2DPlotModel, double, double) may expose internal representation by storing an externally mutable object into WWZPlotPane.chart

new org.aavso.tools.vstar.ui.dialog.period.wwz.WWZPlotPane(JFreeChart, WWZ2DPlotModel, double, double) may expose internal representation by storing an externally mutable object into WWZPlotPane.model

new org.aavso.tools.vstar.ui.dialog.period.wwz.WeightedWaveletZTransformResultDialog(String, String, WeightedWaveletZTransform, WWZCoordinateType) may expose internal representation by storing an externally mutable object into WeightedWaveletZTransformResultDialog.wwt

new org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementDialog(PluginManager) may expose internal representation by storing an externally mutable object into PluginManagementDialog.manager

Unread field: org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementDialog.DIALOG_TITLE; should this field be static?

Unread field: org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementDialog.DIALOG_TITLE_MOD; should this field be static?

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementOperation.getManager() may expose internal representation by returning PluginManagementOperation.manager

new org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManagementOperation(PluginManager, String) may expose internal representation by storing an externally mutable object into PluginManagementOperation.manager

Found reliance on default encoding in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.retrieveRemotePluginInfo(String): new java.io.InputStreamReader(InputStream)

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getLibs() may expose internal representation by returning PluginManager.libs

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getLocalDescriptionsToJarName() may expose internal representation by returning PluginManager.localDescriptions

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getLocalPluginsByJarName() may expose internal representation by returning PluginManager.localPlugins

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getRemoteDescriptionsToJarName() may expose internal representation by returning PluginManager.remoteDescriptions

org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.getRemotePluginsByJarName() may expose internal representation by returning PluginManager.remotePlugins

Possible null pointer dereference in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.deleteAllPlugins() due to return value of called method

Possible null pointer dereference in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.deleteAllPlugins() due to return value of called method

Possible null pointer dereference in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.retrieveLocalPluginInfo() due to return value of called method

Possible null pointer dereference in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.retrieveLocalPluginInfo() due to return value of called method

Exceptional return value of java.io.File.mkdir() ignored in org.aavso.tools.vstar.ui.dialog.plugin.manager.PluginManager.installPlugin(String, PluginManager$Operation)

Should org.aavso.tools.vstar.ui.dialog.prefs.ChartPropertiesSelectionPane$ChartFontLabel be a _static_ inner class?

Should org.aavso.tools.vstar.ui.dialog.prefs.ChartPropertiesSelectionPane$ColorRectComponent be a _static_ inner class?

Potentially ambiguous invocation of either an outer or inherited method org.aavso.tools.vstar.ui.dialog.AbstractOkCancelDialog.createButtonPane() in new org.aavso.tools.vstar.ui.dialog.prefs.ChartPropertiesSelectionPane$FontDialog(ChartPropertiesSelectionPane, Font)

Should org.aavso.tools.vstar.ui.dialog.prefs.ChartPropertiesSelectionPane$FontDialog be a _static_ inner class?

new org.aavso.tools.vstar.ui.dialog.prefs.LocaleSelectionPane() makes inefficient use of keySet iterator instead of entrySet iterator

org.aavso.tools.vstar.ui.dialog.prefs.NewGroupWithStarsDialog.getStarList() may expose internal representation by returning NewGroupWithStarsDialog.starList

Public static org.aavso.tools.vstar.ui.dialog.prefs.PreferencesDialog.getInstance() may expose internal representation by returning PreferencesDialog.instance

Class (org.aavso.tools.vstar.ui.dialog.prefs.PreferencesDialog) using singleton design pattern directly or indirectly implements Serializable interface.

org.aavso.tools.vstar.ui.dialog.prefs.SeriesColorSelectionPane.getChangedSeriesColorMap() may expose internal representation by returning SeriesColorSelectionPane.changedSeriesColorMap

org.aavso.tools.vstar.ui.dialog.prefs.SeriesSizeSelectionPane.getChangedSeriesSizeMap() may expose internal representation by returning SeriesSizeSelectionPane.changedSeriesSizeMap

Should org.aavso.tools.vstar.ui.dialog.prefs.SeriesSizeSelectionPane$DotComponent be a _static_ inner class?

org.aavso.tools.vstar.ui.dialog.series.AbstractSeriesVisibilityDialog.getNextPane() may expose internal representation by returning AbstractSeriesVisibilityDialog.nextPane

new org.aavso.tools.vstar.ui.dialog.series.AbstractSeriesVisibilityDialog(ISeriesInfoProvider, String, JPanel) may expose internal representation by storing an externally mutable object into AbstractSeriesVisibilityDialog.nextPane

Unwritten public or protected field: org.aavso.tools.vstar.ui.dialog.series.AbstractSeriesVisibilityDialog.seriesVisibilityPane

new org.aavso.tools.vstar.ui.dialog.series.MeanSourcePane(ObservationAndMeanPlotModel, ObservationAndMeanPlotPane) may expose internal representation by storing an externally mutable object into MeanSourcePane.obsPlotModel

new org.aavso.tools.vstar.ui.dialog.series.MeanSourcePane(ObservationAndMeanPlotModel, ObservationAndMeanPlotPane) may expose internal representation by storing an externally mutable object into MeanSourcePane.obsPlotPane

new org.aavso.tools.vstar.ui.dialog.series.MultipleSeriesSelectionDialog(SeriesVisibilityPane) may expose internal representation by storing an externally mutable object into MultipleSeriesSelectionDialog.seriesVisibilityPane

org.aavso.tools.vstar.ui.dialog.series.SeriesVisibilityPane.getVisibilityDeltaMap() may expose internal representation by returning SeriesVisibilityPane.visibilityDeltaMap

new org.aavso.tools.vstar.ui.dialog.series.SeriesVisibilityPane(ObservationAndMeanPlotModel, AnalysisType, boolean, boolean, boolean) may expose internal representation by storing an externally mutable object into SeriesVisibilityPane.obsPlotModel

org.aavso.tools.vstar.ui.dialog.series.SeriesVisibilityPane.seriesVisibilityChange(Map) makes inefficient use of keySet iterator instead of entrySet iterator

Switch statement found in org.aavso.tools.vstar.ui.model.list.AAVSOFormatRawDataColumnInfoSource.getTableColumnValue(int, ValidObservation) where default case is missing

Exception thrown in class org.aavso.tools.vstar.ui.model.list.AbstractModelObservationTableModel at new org.aavso.tools.vstar.ui.model.list.AbstractModelObservationTableModel(List, SeriesType) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.model.list.AbstractSyntheticObservationTableModel.getObs() may expose internal representation by returning AbstractSyntheticObservationTableModel.obs

new org.aavso.tools.vstar.ui.model.list.AbstractSyntheticObservationTableModel(List) may expose internal representation by storing an externally mutable object into AbstractSyntheticObservationTableModel.obs

org.aavso.tools.vstar.ui.model.list.ArbitraryFormatRawDataColumnInfoSource.COLUMN_NAMES is a mutable collection which should be package protected

new org.aavso.tools.vstar.ui.model.list.InvalidObservationTableModel(List) may expose internal representation by storing an externally mutable object into InvalidObservationTableModel.invalidObservations

Switch statement found in org.aavso.tools.vstar.ui.model.list.InvalidObservationTableModel.getColumnClass(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.InvalidObservationTableModel.getColumnName(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.InvalidObservationTableModel.getValueAt(int, int) where default case is missing

org.aavso.tools.vstar.ui.model.list.PeriodAnalysisDataTableModel.getData() may expose internal representation by returning PeriodAnalysisDataTableModel.data

new org.aavso.tools.vstar.ui.model.list.PeriodAnalysisDataTableModel(PeriodAnalysisCoordinateType[], Map) may expose internal representation by storing an externally mutable object into PeriodAnalysisDataTableModel.columnTypes

new org.aavso.tools.vstar.ui.model.list.PeriodAnalysisDataTableModel(PeriodAnalysisCoordinateType[], Map) may expose internal representation by storing an externally mutable object into PeriodAnalysisDataTableModel.data

org.aavso.tools.vstar.ui.model.list.PeriodAnalysisDataTableModel.setData(Map) may expose internal representation by storing an externally mutable object into PeriodAnalysisDataTableModel.data

Switch statement found in org.aavso.tools.vstar.ui.model.list.PhasePlotMeanObservationTableModel.getColumnClass(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.PhasePlotMeanObservationTableModel.getColumnName(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.PhasePlotMeanObservationTableModel.getValueAt(int, int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.PhasePlotModelObservationTableModel.getColumnClass(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.PhasePlotModelObservationTableModel.getColumnName(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.PhasePlotModelObservationTableModel.getValueAt(int, int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.RawDataMeanObservationTableModel.getColumnClass(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.RawDataMeanObservationTableModel.getColumnName(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.RawDataMeanObservationTableModel.getValueAt(int, int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.RawDataModelObservationTableModel.getColumnClass(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.RawDataModelObservationTableModel.getColumnName(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.RawDataModelObservationTableModel.getValueAt(int, int) where default case is missing

org.aavso.tools.vstar.ui.model.list.SimpleFormatRawDataColumnInfoSource.COLUMN_NAMES is a mutable collection which should be package protected

Switch statement found in org.aavso.tools.vstar.ui.model.list.SimpleFormatRawDataColumnInfoSource.getTableColumnClass(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.SimpleFormatRawDataColumnInfoSource.getTableColumnTitle(int) where default case is missing

Switch statement found in org.aavso.tools.vstar.ui.model.list.SimpleFormatRawDataColumnInfoSource.getTableColumnValue(int, ValidObservation) where default case is missing

org.aavso.tools.vstar.ui.model.list.ValidObservationTableModel.getObsInserter() may expose internal representation by returning ValidObservationTableModel.obsInserter

org.aavso.tools.vstar.ui.model.list.ValidObservationTableModel.getObsSourceListMap() may expose internal representation by returning ValidObservationTableModel.obsSourceListMap

org.aavso.tools.vstar.ui.model.list.ValidObservationTableModel.getObservations() may expose internal representation by returning ValidObservationTableModel.validObservations

new org.aavso.tools.vstar.ui.model.list.ValidObservationTableModel(Map, List, ITableColumnInfoSource) may expose internal representation by storing an externally mutable object into ValidObservationTableModel.obsSourceListMap

org.aavso.tools.vstar.ui.model.list.WWZDataTableModel.getStats() may expose internal representation by returning WWZDataTableModel.stats

org.aavso.tools.vstar.ui.model.list.WWZDataTableModel.getWwt() may expose internal representation by returning WWZDataTableModel.wwt

new org.aavso.tools.vstar.ui.model.list.WWZDataTableModel(List, WeightedWaveletZTransform) may expose internal representation by storing an externally mutable object into WWZDataTableModel.stats

new org.aavso.tools.vstar.ui.model.list.WWZDataTableModel(List, WeightedWaveletZTransform) may expose internal representation by storing an externally mutable object into WWZDataTableModel.wwt

org.aavso.tools.vstar.ui.model.plot.ContinuousModelFunction.getFit() may expose internal representation by returning ContinuousModelFunction.fit

org.aavso.tools.vstar.ui.model.plot.ContinuousModelFunction.getPpModel() may expose internal representation by returning ContinuousModelFunction.ppModel

new org.aavso.tools.vstar.ui.model.plot.ContinuousModelFunction(UnivariateRealFunction, List, double, ICoordSource) may expose internal representation by storing an externally mutable object into ContinuousModelFunction.fit

org.aavso.tools.vstar.ui.model.plot.ContinuousModelFunction.setPpModel(PhasedObservationAndMeanPlotModel) may expose internal representation by storing an externally mutable object into ContinuousModelFunction.ppModel

new org.aavso.tools.vstar.ui.model.plot.ContinuousModelPlotModel(ContinuousModelFunction) may expose internal representation by storing an externally mutable object into ContinuousModelPlotModel.modelFunction

Integral division result cast to double or float in new org.aavso.tools.vstar.ui.model.plot.ContinuousModelPlotModel(ContinuousModelFunction)

org.aavso.tools.vstar.ui.model.plot.JDCoordSource.instance isn't final but should be

Exception thrown in class org.aavso.tools.vstar.ui.model.plot.ObservationAndMeanPlotModel at new org.aavso.tools.vstar.ui.model.plot.ObservationAndMeanPlotModel(Map, ICoordSource, Comparator, ITimeElementEntity, Map) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.

org.aavso.tools.vstar.ui.model.plot.ObservationAndMeanPlotModel.getMeanObsList() may expose internal representation by returning ObservationAndMeanPlotModel.meanObsList

org.aavso.tools.vstar.ui.model.plot.ObservationAndMeanPlotModel.getMeansChangeNotifier() may expose internal representation by returning ObservationAndMeanPlotModel.meansChangeNotifier

org.aavso.tools.vstar.ui.model.plot.ObservationAndMeanPlotModel.getTimeElementEntity() may expose internal representation by returning ObservationAndMeanPlotModel.timeElementEntity

new org.aavso.tools.vstar.ui.model.plot.ObservationAndMeanPlotModel(Map, ICoordSource, Comparator, ITimeElementEntity, Map) may expose internal representation by storing an externally mutable object into ObservationAndMeanPlotModel.timeElementEntity

org.aavso.tools.vstar.ui.model.plot.ObservationAndMeanPlotModel.determineMeanSeriesSource() makes inefficient use of keySet iterator instead of entrySet iterator

new org.aavso.tools.vstar.ui.dialog.series.SingleSeriesSelectionDialog(ObservationAndMeanPlotModel) may expose internal representation by storing an externally mutable object into SingleSeriesSelectionDialog.obsPlotModel

new org.aavso.tools.vstar.ui.dialog.series.SingleSeriesSelectionPane(ObservationAndMeanPlotModel) may expose internal representation by storing an externally mutable object into SingleSeriesSelectionPane.obsPlotModel

org.aavso.tools.vstar.ui.mediator.DocumentManager.getStatsInfo() may expose internal representation by returning DocumentManager.statsInfo

Write to static field org.aavso.tools.vstar.ui.mediator.DocumentManager.filterNum from instance method org.aavso.tools.vstar.ui.mediator.DocumentManager.getNextUntitledFilterName()

org.aavso.tools.vstar.ui.mediator.Mediator.quit() invokes System.exit(...), which shuts down the entire virtual machine