DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43736>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43736





------- Additional Comments From [EMAIL PROTECTED]  2007-10-30 17:12 -------
Minimal fix would be something like:

===============================================================
====
--- src/main/java/org/apache/log4j/xml/XMLDecoder.java  (revision 570571)
+++ src/main/java/org/apache/log4j/xml/XMLDecoder.java  (working copy)
@@ -183,9 +183,9 @@
     if (owner != null) {
       reader = new LineNumberReader(new InputStreamReader(
               new ProgressMonitorInputStream(owner,
-                      "Loading " + url , url.openStream())));
+                      "Loading " + url , url.openStream(), "UTF-8")));
     } else {
-      reader = new LineNumberReader(new InputStreamReader(url.openStream()));
+      reader = new LineNumberReader(new InputStreamReader(url.openStream(), 
"UTF-8"));
     }
 
     Vector v = new Vector();


With comparable changes likely for UtilLogXMLDecoder.  That isn't sufficient to 
handle UTF-16 encoded 
log files, but would be a good interim fix.  Proper fix probably means 
rewriting the whole thing (likely 
using a SAX instead of a DOM parser to reduce memory use).  Would be good to 
get unit tests around it 
first before a rewrite.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to