Re: Review Request 19616: Added check for null return.

[Daan Hoogland]

On Fri, Apr 11, 2014 at 9:50 AM, Alex Hitchins <a...@alexhitchins.com> wrote:
>
>       if(snapshotInfoReturn.equals(null)){
>         throw new ResourceAllocationException("Take snapshot for VolumeId: "
> + volumeId + " failed.", ResourceType.snapshot);
>       }


This will never throw a ResourceAllocationException. It will throw a
NullPointerException or pass, as you call a method on an Object to see
if it is null (no object).

-- 
Daan