Tal Nisan has submitted this change and it was merged. Change subject: webadmin: remove redundant variable ......................................................................
webadmin: remove redundant variable Remove isNew variable from StorageListModel because it's not needed. The evaluated statement storage==null is always false, because couple of rows before that storage name and descriptions are evaluated, and if storage was null, then NPE would be thrown there. Thus, storage is not null and the expression is always false, and isNew variable is not needed. Change-Id: Ib755a8fb39bae04349ee2617d556a589ab7cfd5c Signed-off-by: Alissa Bonas <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12063 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib755a8fb39bae04349ee2617d556a589ab7cfd5c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alissa Bonas <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
