commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebb (JIRA)" <j...@apache.org>
Subject [jira] Created: (CONFIGURATION-421) VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null
Date Tue, 08 Jun 2010 12:33:11 GMT
VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null
-------------------------------------------------------------------------------------

                 Key: CONFIGURATION-421
                 URL: https://issues.apache.org/jira/browse/CONFIGURATION-421
             Project: Commons Configuration
          Issue Type: Bug
          Components: File reloading
            Reporter: Sebb


VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null:

{noformat}
        if (configuration.getURL() == null && configuration.getFileName() == null)
        {
            return;
        }
        if (this.configuration == null)
        {
            throw new IllegalStateException("No configuration has been set for this strategy");
        }
{noformat}

The if statements should be swapped.

[Note found by Findbugs]

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message