John,

Don't worry about CVS. Good coding skills are much more valuable. I am looking forward 
to your contribution. Ceki 

At 14:29 12.04.2001 -0400, you wrote:
>Ceki, 
>
>When I get a chance I'll clean up the DebugProxy class we have here (rename
>it to MethodTracer? maybe?)  I'll try to comment it fairly heavily and point
>out the short comings that it has.  Give me a week or so.  Umm, shamed to
>admit it, but I'm CVS illiterate, guess I'll have to learn, so be prepared
>for silly questions and fumbled attempts :-)
>
>John Volkar
>
>-----Original Message-----
>From: Ceki Gülcü [mailto:[EMAIL PROTECTED]]
>Sent: Wednesday, April 11, 2001 12:42 PM
>To: LOG4J Users Mailing List
>Subject: RE: wrapper for Log4j for method entrance/exit?
>
>
>At 11:54 22.03.2001 -0500, John wrote:
>>Ceki, 
>>So far as I've seen (aside from AspectJ, which I shied away from because of
>>the precompiler "voodoo" nature) there is not any *totally* non intrusive
>>way.  
>>
>>If you do not have the luxory of a factory that you can do the constructor
>>interception at, you can do a search and replace style thingie where you:
>>
>>Replace:        "= new " + * + ");"
>>with:           "= new DebugProxy.for( " + * + ") );"
>>
>>So:             FooBarClass fb = new FooBarClass(myargs);
>>Becomes:        FooBarClass fb = DebugProxy.for( new FooBarClass(myargs) );
>>
>>And the DebugProxy class is a static class whose .for() method accepts an
>>object, wraps it in a Proxy and then returns it. (typically only wrapping
>it
>>if a debug flag is set.)
>>
>>Unfortunatly this uglifies your code a bit, but if you do not have a
>>factory...  You can tweak the DebugProxy.for method to only wrap certain
>>classes, etc.
>>
>>In a way I'd like to see something like this added to log4j's varia
>package.
>>I mean it *is* kinda neat (and useful too!)
>
>John,
>
>OK. So why don't you scratch that itch? I will gladly add your contribution
>to the log4j distrib. Ceki
>
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: [EMAIL PROTECTED]
>For additional commands, e-mail: [EMAIL PROTECTED]
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: [EMAIL PROTECTED]
>For additional commands, e-mail: [EMAIL PROTECTED]

--
Ceki Gülcü     Web: http://qos.ch 
email: [EMAIL PROTECTED] or [EMAIL PROTECTED]


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

Reply via email to