Change in ovirt-engine[master]: restapi: Use DiskMapper.map static method instead of constru...

[juan.hernandez at redhat.com]

Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Use DiskMapper.map static method instead of constructor
......................................................................


restapi: Use DiskMapper.map static method instead of constructor

This is needed to avoid the following findbugs complaint:

  Return value of method without side effect is ignored This code calls
  a method and ignores the return value. However our analysis shows that
  the method (including its implementations in subclasses if any) does not
  produce any effect other than return value.  Thus this call can be
  removed.  We are trying to reduce the false positives as much as
  possible, but in some cases this warning might be wrong. Common
  false-positive cases include:

  - The method is designed to be overridden and produce a side effect in
  other projects which are out of the scope of the analysis.

  - The method is called to trigger the class loading which may have a
  side effect.

  - The method is called just to get some exception.

  If you feel that our assumption is incorrect, you can use a
  @CheckReturnValue annotation to instruct FindBugs that ignoring the
  return value of this method is acceptable.

  Bug kind and pattern: RV - RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT

Change-Id: Iaa35254aa8030baef4a3b7b48afeb699993ac606
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDiskAttachmentsResource.java
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/57778
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa35254aa8030baef4a3b7b48afeb699993ac606
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>