diff --git a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/validators/ValidatorUniqueString.java b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/validators/ValidatorUniqueString.java index 73b2c85a065..1ac092497cb 100644 --- a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/validators/ValidatorUniqueString.java +++ b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/validators/ValidatorUniqueString.java @@ -11,7 +11,9 @@ * Emily Bruner, Mazen Faraj, Adrian Storisteanu, Li Ding, and Kent Hawley. * * Contributors: - * {Name} (company) - description of contribution. + * David Dykstal (IBM) - generalized Vector argumnents to Collections + * Kevin Doyle (IBM) - 174776: perform required sorting of Collection arguments + * David Dykstal (IBM) - 174776: disallowed sorting of input arguments, used copies ********************************************************************************/ package org.eclipse.rse.ui.validators; @@ -40,6 +42,8 @@ public class ValidatorUniqueString public static final boolean CASE_SENSITIVE = true; public static final boolean CASE_INSENSITIVE = false; public static final char QUOTE = '\"'; + private static final String[] EMPTY_LIST = new String[0]; + protected ISystemValidator syntaxValidator; protected boolean caseSensitive; protected boolean useUpperCase; @@ -132,6 +136,7 @@ public class ValidatorUniqueString { existingList = new String[newList.size()]; newList.toArray(existingList); + init(existingList, caseSensitive); } } /** @@ -156,27 +161,26 @@ public class ValidatorUniqueString /** * Initialize sorted array. */ - private void init(String existingList[], boolean caseSensitive) - { - this.existingList = existingList; - if (existingList == null) - return; - if (!caseSensitive) // speed up comparison by converting to all lowercase - { - String newList[] = new String[existingList.length]; - for (int idx=0; idx