You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@harmony.apache.org by Andrew Zhang <zh...@gmail.com> on 2006/08/30 05:48:26 UTC

Re: [classlib][logging]yet another non-bug difference with RI(Re: [jira] Created: (HARMONY-1300) [classlib][logging] XMLFormatter.format returns a String contains "" while RI doesn't when log message and pattern is null.)

On 8/30/06, Paulex Yang <pa...@gmail.com> wrote:

> From the DTD for j.u.logging.XMLFormatter[1], message is required by
> record, so I consider this is RI's bug, and I suggest to follow spec in
> this case, because it is hard to believe this difference with RI will
> make application fail, while there is possibilities for some
> applications to validate logging record using the DTD, it may fail on
> RI. If no one objects, I will mark this issue as "non-bug difference
> with RI".


 Agreed.

Paulex,  please don't apply harmony-1300 patch, and close it as "non-bug
difference with RI".

[1]
> ...
> <!-- Each logging call is described by a record element. -->
> <!ELEMENT record (date, millis, sequence, logger?, level,class?,
> method?, thread?, message, key?, catalog?, param*, exception?)>
> ...
>
> Andrew Zhang (JIRA) wrote:
> > [classlib][logging] XMLFormatter.format returns a String contains
> "<message/>" while RI doesn't when log  message and pattern is null.
> >
> ---------------------------------------------------------------------------------------------------------------------------------------
> >
> >                  Key: HARMONY-1300
> >                  URL: http://issues.apache.org/jira/browse/HARMONY-1300
> >              Project: Harmony
> >           Issue Type: Bug
> >           Components: Classlib
> >             Reporter: Andrew Zhang
> >
> >
> > Following test reproduces the bug:
> >       public void test_format() {
> >               formatter = new XMLFormatter();
> >               LogRecord lr = new LogRecord(Level.SEVERE, null);
> >               String output = formatter.format(lr);
> >               assertFalse(output.indexOf("message") > 0);
> >       }
> > The test passes against RI while fails against Harmony.
> >
> > I'll upload a patch to fix this problem soon.
> >
> > Best regards,
> > Andrew
> >
> >
>
>
> --
> Paulex Yang
> China Software Development Lab
> IBM
>
>
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>


-- 
Andrew Zhang
China Software Development Lab, IBM