Vojtech Szocs has posted comments on this change. Change subject: webadmin: Avoid NPE in Enum Translator (#802370) ......................................................................
Patch Set 1: Verified; Looks good to me, approved Note: the Translator class isn't used directly in oVirt, just its subclass EnumTranslator. Maybe in future we can make Translator an interface, instead of extending HashMap directly with all its methods. -- To view, visit http://gerrit.ovirt.org/3079 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4b3701de53c2533ccd4d0e528725a646cbb45589 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchap...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
