At 08:01 28.07.2001 -0400, Geir Magnusson Jr. wrote:
>Ceki Gülcü wrote:
>> 
>> Geir, Sam,
>> 
>> This is to be expected. The new version of log4j (still under
>> development) has removed some methods deprecated in 1.1. These methods
>> are
>> 
>>   setOption(String key, String value),
>>   getOptionStrings()
>> 
>> and related option constants which were defined in all appenders and layouts.
>> 
>> In log4j 1.2, we moved to JavaBeans type of configuration using
>> introspection and these methods and constants are no longer required.

Sorry, the above statement is incorrect. It is in log4j 1.1 that  we moved to 
JavaBeans type of configuration using introspection. In log4j 1.2 deprecated methods 
will be (are) removed.  

>> Compatibility with both log4j 1.1 and future 1.2 should be easy to achieve. If an 
>option
>> was called "Toto", then you should call:
>> 
>>   appender.setToto(optionValue);
>> 
>> instead of the previous:
>> 
>>   appender.setOption(TOTO_OPTION, optionValue);
>> 
>> For example, replace:
>> 
>>   appender.setOption(SMTPAppender.SMTP_HOST_OPTION, smtpHost);
>> 
>> with
>> 
>>   appender.setSMTPHost(smtpHost);
>
>Sounds good.
> 
>> The case with setMaxFileSize in RollingFileAppender is unique. Without
>> delving into the details, I would like to ask you to call
>> setMaximumFileSize(long maxFileSize) instead of setMaxFileSize(long).
>
>Ok - will do.  I will assume this was deprecated, as I am sure you live
>in fear of the Deprecation Goon Squad as much as we do... :)
> 
>> Let me know if you have any difficulty in making these
>> changes. Regards, Ceki
>
>I will try to get them in right now... I will be away tonight and
>tomorrow, but will be able to verify if all went well when I get back -
>I guess I can build against the latest log4j to test, too.

Yes. These changes are compatible with log4j 1.1.x and the latest code in our CVS rep 
(future log4j 1.2).

>Thanks for the prompt response.

You are welcome. Ceki


--
Ceki Gülcü - http://qos.ch


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

Reply via email to