Tal Nisan has posted comments on this change. Change subject: core: VdcActionUtils: Unwrap else blocks ......................................................................
Patch Set 2: Code-Review+2 Looks ok basically but I'd rewrite this method totally, I don't understand why there is a check on the class name string if an instanceof check is performed and even if there is some hidden logic behind checking the class name, why use endsWith and not equals? -- To view, visit http://gerrit.ovirt.org/18852 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I475e1c4a99ba004bc3ebe2182a7351a0d28dcbda Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
