Oliver, Many thanks for spontaneously taking care of this bug. I appreciate it very much.
One question though. Shouldn't you commit this fix to the 1.2 branch as the CVS trunk is for log4j 1.3 development only? The command to access the v1_2-branch is: cvs -d XYZ checkout -r v1_2-branch jakarta-log4j where XYZ is the remote repository name, that is ":ext:[EMAIL PROTECTED]:/home/cvs" for me. Once you have checked out the 1.2 branch, you should find a file called BRANCHES with more detailed info on... branches. Many thanks in advance, Ceki ps: Please do not hesitate to contact me if you encounter any problems. At 12:38 25.05.2002 +0000, [EMAIL PROTECTED] wrote: >oburn 02/05/25 05:38:03 > > Modified: src/java/org/apache/log4j/chainsaw XMLFileHandler.java > LoggingReceiver.java > Log: > PR: 9268 > Fixed the bug in the code that loads events from an XML file. The code now > correctly handles multiple calls to characters(). Tested with Crimson and > Xerces under JDK1.3.1 and JDK1.4. > Also fixed Checkstyle errors. > > Revision Changes Path > 1.3 +19 > -26 jakarta-log4j/src/java/org/apache/log4j/chainsaw/XMLFileHandler.java > > Index: XMLFileHandler.java > =================================================================== > RCS file: > /home/cvs/jakarta-log4j/src/java/org/apache/log4j/chainsaw/XMLFileHandler.java,v > retrieving revision 1.2 > retrieving revision 1.3 > diff -u -r1.2 -r1.3 > --- XMLFileHandler.java 2 May 2002 11:25:49 -0000 1.2 > +++ XMLFileHandler.java 25 May 2002 12:38:02 -0000 1.3 > @@ -37,8 +37,6 @@ > private final MyTableModel mModel; > /** the number of events in the document **/ > private int mNumEvents; > - /** the current element being parsed **/ > - private String mCurrentElement; [snip] -- Ceki -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
