Hi,
Well, the ThrowableClassMatchFilter relied directly on the getThrowable method of 
org.apache.log4j.spi.ThrowableInformation ;)  Given that this method is gone for good, 
we have two choices:
- Discard the ThrowableClassMatchFilter altogether
- Add a getThrowableClassName or getThrowableClass method to ThrowableInformation and 
attempt to make the ThrowableClassMatchFilter work with this method instead of 
getThrowable.  I think I can make it work this way.

I added this filter a few months ago because it was such and easy feature that I think 
has considerable real-world value.  But given that it's in the sandbox, we can safely 
assume only a few people know about it and even fewer have used it, so we can discard 
it if we want ;)

Yoav Shapira
Millennium ChemInformatics


>-----Original Message-----
>From: Ceki Gülcü [mailto:[EMAIL PROTECTED]
>Sent: Tuesday, December 09, 2003 10:24 AM
>To: Log4J Developers List
>Subject: Re: Sandbox problems
>
>At 08:28 PM 12/8/2003 -0500, you wrote:
>>Raymond DeCampo wrote:
>>>Ceki Gülcü wrote:
>>>
>>>>Given my recent changes to ThrowableInformation the following files no
>>>>longer compile properly.
>>>>
>>>>The method getThrowable() is undefined for the type ThrowableInformation
>>>>ThrowableClassMatchFilter.java
>>>>log4j-sandbox/src/java/org/apache/log4j/filter  line 170
>
>>>>The method getThrowable() is undefined for the type ThrowableInformation
>>>>PreparedStatementParameter.java
>>>>log4j-sandbox/src/java/org/apache/log4j/jdbc    line 189
>>>Ceki,
>>>I'll take a look.
>>>Ray
>>
>>I've checked in changes for PreparedStatementAppender.java and
>>PreparedStatementParameter.java.  I didn't notice the
>>ThrowableClassMatchFilter.java class when I volunteered earlier.
>
>Thank for you for updating the PreparedStatementAppender and
>PreparedStatementParameter classes.
>
>>  I don't have time now, since I'm not familiar with that class, but if
>> nobody has tackled it by this time tomorrow I will take a look.
>
>Perhaps Yoav could have a look as well. Yoav?
>
>>Ray
>
>--
>Ceki Gülcü
>
>      For log4j documentation consider "The complete log4j manual"
>      ISBN: 2970036908 http://www.qos.ch/shop/products/clm_t.jsp
>
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: [EMAIL PROTECTED]
>For additional commands, e-mail: [EMAIL PROTECTED]




This e-mail, including any attachments, is a confidential business communication, and 
may contain information that is confidential, proprietary and/or privileged.  This 
e-mail is intended only for the individual(s) to whom it is addressed, and may not be 
saved, copied, printed, disclosed or used by anyone else.  If you are not the(an) 
intended recipient, please immediately delete this e-mail from your computer system 
and notify the sender.  Thank you.


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

Reply via email to