Moti Asayag has posted comments on this change.

Change subject: utils: fix compiler warnings (unit test)
......................................................................


Patch Set 4: Verified+1

(1 comment)

http://gerrit.ovirt.org/#/c/33739/4/backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/linq/LinqUtilsTest.java
File 
backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/linq/LinqUtilsTest.java:

Line 14:     private List<String> list;
Line 15:     private List<String> cutlist;
Line 16: 
Line 17:     public LinqUtilsTest() {
Line 18:         list  = new LinkedList<String>();
java 7 syntax allows not to re-declare the type.
  list  = new LinkedList<>();

could be done in a separate patch though.
Line 19:         list.add("one");
Line 20:         list.add("two");
Line 21:         list.add("three");
Line 22: 


-- 
To view, visit http://gerrit.ovirt.org/33739
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b9b149dc43bc0d60cf0bbfcb530f642d4128820
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tim Speetjens <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Tim Speetjens <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to