harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Geir Magnusson Jr." <g...@pobox.com>
Subject Re: [classlib][logging]yet another non-bug difference with RI(Re: [jira] Created: (HARMONY-1300) [classlib][logging] XMLFormatter.format returns a String contains "<message/>" while RI doesn't when log message and pattern is null.)
Date Wed, 30 Aug 2006 03:32:34 GMT
agreed

Paulex Yang 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".
> 
> [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
>>
>>   
> 
> 


---------------------------------------------------------------------
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


Mime
View raw message