Change in ovirt-engine[master]: core: Use "Integer.compare" in "InitDictinoaries"

[juan.hernandez at redhat.com]

Juan Hernandez has submitted this change and it was merged.

Change subject: core: Use "Integer.compare" in "InitDictinoaries"
......................................................................


core: Use "Integer.compare" in "InitDictinoaries"

This is needed in order to avoid the following findbugs error:

  Boxing a primitive to compare

  A boxed primitive is created just to call compareTo method. It's more
  efficient to use static compare method (for double and float since
  Java 1.4, for other primitive types since Java 1.7) which works on
  primitives directly.

  Bug kind and pattern: Bx - DM_BOXED_PRIMITIVE_FOR_COMPARE

Change-Id: I8111481cf815abc0aa4da66a0cf47b8cb72b3a9b
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CpuFlagsManagerHandler.java
1 file changed, 1 insertion(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8111481cf815abc0aa4da66a0cf47b8cb72b3a9b
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: automation at ovirt.org