commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Luc Maisonobe (JIRA)" <j...@apache.org>
Subject [jira] Updated: (EXEC-35) inefficiencies identified by findbugs
Date Mon, 05 Jan 2009 22:01:44 GMT

     [ https://issues.apache.org/jira/browse/EXEC-35?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Luc Maisonobe updated EXEC-35:
------------------------------

    Attachment: findbugs-configuration.patch

patch to set up findbugs and avoid some false positives

> inefficiencies identified by findbugs
> -------------------------------------
>
>                 Key: EXEC-35
>                 URL: https://issues.apache.org/jira/browse/EXEC-35
>             Project: Commons Exec
>          Issue Type: Bug
>    Affects Versions: 1.0
>         Environment: linux
>            Reporter: Luc Maisonobe
>            Priority: Trivial
>         Attachments: findbugs-configuration.patch, findbugs-fixes.patch
>
>
> I've run findbugs, it shows the following issues:
> DefaultExecutor (line 203)
>   setExitValues(int[]) may expose internal representation by storing an
>   externally mutable object into DefaultExecutor.exitValues 	
> DefaultProcessingEnvironment (lines 169 and 171)
>   Hard coded reference to an absolute pathname
> DefaultProcessingEnvironment (line 108) and
> OpenVmsProcessingEnvironment (line 89)
>   concatenates strings using + in a loop
> OpenVmsProcessingEnvironment (line 114) and
> MapUtils (line 72)
>   inefficient use of keySet iterator instead of entrySet iterator
> The first issue should probably be fixed by cloning the array, but does
> not appear to be an important problem. The second issue is a false
> positive (if findbugs were activated by default in the pom, this could
> be prevented by a configured exclusion file). The remaining issues are
> harmless.
> I will provide a patch to fix these issues shortly

-- 
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