Change in ovirt-engine[master]: restapi: findbugs error

masayag at redhat.com masayag at redhat.com
Mon May 13 08:38:03 UTC 2013


Moti Asayag has submitted this change and it was merged.

Change subject: restapi: findbugs error
......................................................................


restapi: findbugs error

Fixing a findbug error although the warned NPE by findbugs
shouldn't occur (there is no code path which will end in NPE),
it is easier to pass its warning by using the 'else' statement.

Change-Id: I8b6749df88b78c3cdb6ef187bebc86de912856b7
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendClusterNetworksResource.java
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/14691
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8b6749df88b78c3cdb6ef187bebc86de912856b7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>



More information about the Engine-commits mailing list