Author: markt Date: Fri Oct 21 08:56:26 2016 New Revision: 1765957 URL: http://svn.apache.org/viewvc?rev=1765957&view=rev Log: Whitespace and Javadoc changes to align with later versions (makes diffs easier when considering backports)
Modified: tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java Modified: tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java?rev=1765957&r1=1765956&r2=1765957&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/res/StringManager.java Fri Oct 21 08:56:26 2016 @@ -14,7 +14,6 @@ * See the License for the specific language governing permissions and * limitations under the License. */ - package org.apache.tomcat.util.res; import java.text.MessageFormat; @@ -40,32 +39,30 @@ import java.util.ResourceBundle; * the package name given plus the suffix of "LocalStrings". In * practice, this means that the localized information will be contained * in a LocalStrings.properties file located in the package - * directory of the classpath. + * directory of the class path. * * <p>Please see the documentation for java.util.ResourceBundle for * more information. * - * - * * @author James Duncan Davidson [dun...@eng.sun.com] * @author James Todd [go...@eng.sun.com] * @author Mel Martinez [mmarti...@g1440.com] * @see java.util.ResourceBundle */ - public class StringManager { // Locale.ROOT isn't available until Java 6 private static final Locale LOCALE_ROOT = new Locale("", "", ""); - + private static int LOCALE_CACHE_SIZE = 10; - + /** * The ResourceBundle for this StringManager. */ private final ResourceBundle bundle; private final Locale locale; + /** * Creates a new StringManager for a given package. This is a * private method and all access to it is arbitrated by the @@ -79,19 +76,19 @@ public class StringManager { ResourceBundle bnd = null; try { bnd = ResourceBundle.getBundle(bundleName, locale); - } catch( MissingResourceException ex ) { + } catch (MissingResourceException ex) { // Try from the current loader (that's the case for trusted apps) // Should only be required if using a TC5 style classloader structure // where common != shared != server ClassLoader cl = Thread.currentThread().getContextClassLoader(); - if( cl != null ) { + if (cl != null) { try { bnd = ResourceBundle.getBundle(bundleName, locale, cl); - } catch(MissingResourceException ex2) { + } catch (MissingResourceException ex2) { // Ignore } } - } + } bundle = bnd; // Get the actual locale, which may be different from the requested one if (bundle != null) { @@ -106,19 +103,21 @@ public class StringManager { } } + /** - Get a string from the underlying resource bundle or return - null if the String is not found. - - @param key to desired resource String - @return resource String matching <i>key</i> from underlying - bundle or null if not found. - @throws IllegalArgumentException if <i>key</i> is null. + * Get a string from the underlying resource bundle or return null if the + * String is not found. + * + * @param key to desired resource String + * + * @return resource String matching <i>key</i> from underlying bundle or + * null if not found. + * + * @throws IllegalArgumentException if <i>key</i> is null */ public String getString(String key) { - if(key == null){ + if (key == null){ String msg = "key may not have a null value"; - throw new IllegalArgumentException(msg); } @@ -133,26 +132,30 @@ public class StringManager { //bad: shouldn't mask an exception the following way: // str = "[cannot find message associated with key '" + key + // "' due to " + mre + "]"; - // because it hides the fact that the String was missing - // from the calling code. - //good: could just throw the exception (or wrap it in another) - // but that would probably cause much havoc on existing - // code. - //better: consistent with container pattern to - // simply return null. Calling code can then do - // a null check. - str = null; + // because it hides the fact that the String was missing + // from the calling code. + //good: could just throw the exception (or wrap it in another) + // but that would probably cause much havoc on existing + // code. + //better: consistent with container pattern to + // simply return null. Calling code can then do + // a null check. + str = null; } return str; } + /** * Get a string from the underlying resource bundle and format * it with the given set of arguments. * - * @param key - * @param args + * @param key The key for the required message + * @param args The values to insert into the message + * + * @return The request string formatted with the provided arguments or the + * key if the key was not found. */ public String getString(final String key, final Object... args) { String value = getString(key); @@ -165,13 +168,17 @@ public class StringManager { return mf.format(args, new StringBuffer(), null).toString(); } + /** - * Identify the Locale this StringManager is associated with + * Identify the Locale this StringManager is associated with. + * + * @return The Locale associated with the StringManager */ public Locale getLocale() { return locale; } + // -------------------------------------------------------------- // STATIC SUPPORT METHODS // -------------------------------------------------------------- @@ -179,18 +186,22 @@ public class StringManager { private static final Map<String,Map<Locale,StringManager>> managers = new Hashtable<String,Map<Locale,StringManager>>(); + /** * Get the StringManager for a particular package. If a manager for * a package already exists, it will be reused, else a new * StringManager will be created and returned. * * @param packageName The package name + * + * @return The instance associated with the given package and the default + * Locale */ - public static final synchronized StringManager getManager( - String packageName) { + public static final StringManager getManager(String packageName) { return getManager(packageName, Locale.getDefault()); } - + + /** * Get the StringManager for a particular package and Locale. If a manager * for a package/Locale combination already exists, it will be reused, else @@ -198,6 +209,8 @@ public class StringManager { * * @param packageName The package name * @param locale The Locale + * + * @return The instance associated with the given package and Locale */ public static final synchronized StringManager getManager( String packageName, Locale locale) { @@ -234,10 +247,13 @@ public class StringManager { return mgr; } + /** * Retrieve the StringManager for a list of Locales. The first StringManager * found will be returned. * + * @param packageName The package for which the StringManager was + * requested * @param requestedLocales the list of Locales * * @return the found StringManager or the default StringManager @@ -254,14 +270,15 @@ public class StringManager { // Return the default return getManager(packageName); } - + + /** * Always returns null. * * @param bundle The resource bundle - * + * * @return Always <code>null</code> - * + * * @deprecated This is unused in Tomcat 6 and will be removed in Tomcat 7 */ @Deprecated --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org