[jboss-svn-commits] JBoss Common SVN: r3351 - jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/handlers.

[jboss-svn-commits at lists.jboss.org]

Author: david.lloyd at jboss.com
Date: 2009-07-08 23:54:28 -0400 (Wed, 08 Jul 2009)
New Revision: 3351

Modified:
   jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/handlers/OutputStreamHandler.java
Log:
Encoding is not a volatile field and has no other protection of any kind - so I guess it is up to us...

Modified: jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/handlers/OutputStreamHandler.java
===================================================================
--- jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/handlers/OutputStreamHandler.java	2009-07-09 03:35:31 UTC (rev 3350)
+++ jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/handlers/OutputStreamHandler.java	2009-07-09 03:54:28 UTC (rev 3351)
@@ -67,6 +67,17 @@
     }
 
     /**
+     * Get the target encoding.
+     *
+     * @return the target encoding, or {@code null} if the platform default is being used
+     */
+    public String getEncoding() {
+        synchronized (outputLock) {
+            return super.getEncoding();
+        }
+    }
+
+    /**
      * Set the target encoding.
      *
      * @param encoding the new encoding
@@ -75,8 +86,8 @@
      */
     public void setEncoding(final String encoding) throws SecurityException, UnsupportedEncodingException {
         // superclass checks access
-        super.setEncoding(encoding);
         synchronized (outputLock) {
+            super.setEncoding(encoding);
             final OutputStream outputStream = this.outputStream;
             updateWriter(outputStream, encoding);
         }