Allon Mureinik has uploaded a new change for review.

Change subject: core: BaseAutoCompleter.getCompletion() array size
......................................................................

core: BaseAutoCompleter.getCompletion() array size

Used the proper way to convert a List to an array in
BaseAutoCompleter.getCompletion(), for better performance.

Change-Id: I0e699a3307e05694f5456282b07e95cf22332d44
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/BaseAutoCompleter.java
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/53/12353/1

diff --git 
a/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/BaseAutoCompleter.java
 
b/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/BaseAutoCompleter.java
index 1ed317e..b7e876e 100644
--- 
a/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/BaseAutoCompleter.java
+++ 
b/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/BaseAutoCompleter.java
@@ -50,8 +50,7 @@
         String[] retval = new String[0];
         if (mVerbCompletion.containsKey(wordPart.toUpperCase())) {
             List<String> curList = mVerbCompletion.get(wordPart.toUpperCase());
-            retval = new String[curList.size()];
-            retval = curList.toArray(new String[] {});
+            retval = curList.toArray(new String[curList.size()]);
         }
         return retval;
     }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0e699a3307e05694f5456282b07e95cf22332d44
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to