In message <94d710af0908240148x56fa02a4rdee99bc022f83...@mail.gmail.com>, Sean Qiu writes: > > I suddenly realized that we had entered code freezing stage. > Shall I revert it back ?
I am just looking at the commit/jira. FYI: it is HARMONY-6301 not 6031. -Mark. > 2009/8/24 <qi...@apache.org>: > > Author: qiuxx > > Date: Mon Aug 24 08:13:13 2009 > > New Revision: 807111 > > > > URL: http://svn.apache.org/viewvc?rev=3D807111&view=3Drev > > Log: > > Apply for HARMONY-6031, There are 2 unused member in java.util.Locale > > > > Modified: > > =A0 =A0harmony/enhanced/classlib/trunk/modules/luni/src/main/java/java/ut= > il/Locale.java > > > > Modified: harmony/enhanced/classlib/trunk/modules/luni/src/main/java/java= > /util/Locale.java > > URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules= > /luni/src/main/java/java/util/Locale.java?rev=3D807111&r1=3D807110&r2=3D807= > 111&view=3Ddiff > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= > =3D=3D=3D=3D > > --- harmony/enhanced/classlib/trunk/modules/luni/src/main/java/java/util/= > Locale.java (original) > > +++ harmony/enhanced/classlib/trunk/modules/luni/src/main/java/java/util/= > Locale.java Mon Aug 24 08:13:13 2009 > > @@ -17,15 +17,12 @@ > > > > =A0package java.util; > > > > -import java.io.File; > > =A0import java.io.IOException; > > =A0import java.io.ObjectInputStream; > > =A0import java.io.ObjectOutputStream; > > =A0import java.io.ObjectStreamField; > > =A0import java.io.Serializable; > > =A0import java.security.AccessController; > > -import java.util.zip.ZipEntry; > > -import java.util.zip.ZipFile; > > > > =A0import org.apache.harmony.luni.util.PriviAction; > > > > @@ -44,8 +41,6 @@ > > > > =A0 =A0 private static final long serialVersionUID =3D 914908174963815063= > 6L; > > > > - =A0 =A0private static volatile Locale[] availableLocales; > > - > > =A0 =A0 // Initialize a default which is used during static > > =A0 =A0 // initialization of the default for the platform. > > =A0 =A0 private static Locale defaultLocale =3D new Locale(); > > @@ -290,91 +285,6 @@ > > =A0 =A0 =A0 =A0 return false; > > =A0 =A0 } > > > > - =A0 =A0static Locale[] find(String prefix) { > > - =A0 =A0 =A0 =A0int last =3D prefix.lastIndexOf('/'); > > - =A0 =A0 =A0 =A0final String thePackage =3D prefix.substring(0, last + 1= > ); > > - =A0 =A0 =A0 =A0int length =3D prefix.length(); > > - =A0 =A0 =A0 =A0final String classPrefix =3D prefix.substring(last + 1, = > length); > > - =A0 =A0 =A0 =A0Set<String> result =3D new HashSet<String>(); > > - =A0 =A0 =A0 =A0StringTokenizer paths =3D new StringTokenizer(System.get= > Property( > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0"org.apache.harmony.boot.class.path", ""= > ), System.getProperty( //$NON-NLS-1$ //$NON-NLS-2$ > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0"path.separator", ";")); //$NON-NLS-1$//= > $NON-NLS-2$ > > - =A0 =A0 =A0 =A0while (paths.hasMoreTokens()) { > > - =A0 =A0 =A0 =A0 =A0 =A0String nextToken =3D paths.nextToken(); > > - =A0 =A0 =A0 =A0 =A0 =A0File directory =3D new File(nextToken); > > - =A0 =A0 =A0 =A0 =A0 =A0if (directory.exists()) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (directory.isDirectory()) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0String path; > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0try { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0path =3D directory.getCa= > nonicalPath(); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} catch (IOException e) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0continue; > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0File newDir; > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (path.charAt(path.length() - = > 1) =3D=3D File.separatorChar) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0newDir =3D new File(path= > + thePackage); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} else { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0newDir =3D new File(path= > + File.separatorChar > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0+ thePac= > kage); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (newDir.isDirectory()) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0String[] list =3D newDir= > .list(); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0for (int i =3D 0; i < li= > st.length; i++) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0String name =3D = > list[i]; > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (name.startsW= > ith(classPrefix) > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0= > && name.endsWith(".class")) { //$NON-NLS-1$ > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0result > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0= > =A0 =A0.add(name.substring(0, > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0= > =A0 =A0 =A0 =A0 =A0 =A0name.length() - 6)); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} else { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0// Handle ZIP/JAR files. > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0try { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0ZipFile zip =3D new ZipF= > ile(directory); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0Enumeration<? extends Zi= > pEntry> entries =3D zip.entries(); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0while (entries.hasMoreEl= > ements()) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0ZipEntry e =3D e= > ntries.nextElement(); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0String name =3D = > e.getName(); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (name.startsW= > ith(prefix) > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0= > && name.endsWith(".class")) {//$NON-NLS-1$ > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0result.a= > dd(name.substring(last + 1, name > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0= > =A0 =A0.length() - 6)); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0zip.close(); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} catch (IOException e) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0// Empty > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0Locale[] locales =3D new Locale[result.size()]; > > - =A0 =A0 =A0 =A0int i =3D 0; > > - =A0 =A0 =A0 =A0for (String name : result) { > > - =A0 =A0 =A0 =A0 =A0 =A0int index =3D name.indexOf('_'); > > - =A0 =A0 =A0 =A0 =A0 =A0int nextIndex =3D name.indexOf('_', index + 1); > > - =A0 =A0 =A0 =A0 =A0 =A0if (nextIndex =3D=3D -1) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0locales[i++] =3D new Locale(name.substri= > ng(index + 1, name > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0.length()), ""); //$NON-= > NLS-1$ > > - =A0 =A0 =A0 =A0 =A0 =A0} else { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0String language =3D name.substring(index= > + 1, nextIndex); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0String variant; > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if ((index =3D name.indexOf('_', nextInd= > ex + 1)) =3D=3D -1) { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0variant =3D ""; //$NON-NLS-1$ > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0index =3D name.length(); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} else { > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0variant =3D name.substring(index= > + 1, name.length()); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0String country =3D name.substring(nextIn= > dex + 1, index); > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0locales[i++] =3D new Locale(language, co= > untry, variant); > > - =A0 =A0 =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0} > > - =A0 =A0 =A0 =A0return locales; > > - =A0 =A0} > > - > > =A0 =A0 =A0 =A0/** > > =A0 =A0 =A0* Gets the list of installed {...@code Locale}. At least a > > {...@code= > Locale} that is equal to > > =A0 =A0 =A0* {...@code Locale.US} must be contained in this array. > > > > > > >
