Eli Mesika has submitted this change and it was merged. Change subject: core: remove redundant code ...................................................................... core: remove redundant code getMethod will never return null. It will either return the method, or will throw an exception. So, in this case, both the if, and the operation inside it, are redundant it will always be false so inner code will never be reached. Change-Id: I878243d9a3e205990d2a029c7443cd6d62feb0d3 Signed-off-by: Oved Ourfali <oourfali@redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/BatchProcedureExecutionConnectionCallback.java 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22072 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I878243d9a3e205990d2a029c7443cd6d62feb0d3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server