Yair Zaslavsky has posted comments on this change. Change subject: core : adding common utilities for VmDevice - VmDeviceCommonUtils ......................................................................
Patch Set 1: (1 inline comment) .................................................... File backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmDeviceCommonUtils.java Line 9: public final static String DISK = "disk"; Are these constants related somehow to each other? Can they be "collected" into an enum? I guess this will effect other places in code. I think using enums where possible is better, but maybe I'm completely wrong here and we cannot use enums for these constants... -- To view, visit http://gerrit.ovirt.org/1163 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie61cccbe2c866997558dc84041fb0c588133f8a8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
