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

Daan Hoogland closed CLOUDSTACK-8586.
-------------------------------------
    Resolution: Fixed

further investigation in the use of null as return values is needed. 
Inconsistency in the semantics of methods arise from the use of null quite 
easily. the issues mentioned are fixed.

> findbugs report on null return for Boolean
> ------------------------------------------
>
>                 Key: CLOUDSTACK-8586
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8586
>             Project: CloudStack
>          Issue Type: Improvement
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>            Reporter: Daan Hoogland
>            Assignee: Daan Hoogland
>
> NP_BOOLEAN_RETURN_NULL says that returning null in a Boolean function is bad 
> practice. It makes sense as it might be used in an if statement and lead to 
> NPE. The functions reported on are isThisOrThat() functions and a possible 
> solution is renaming them to getThisOrThat().
> A small further investigation of the solution and the semantics of the 
> present implementation is required.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to