We will definately take your points into consideration, as they are quite
valid.  In fact, probably right now Chainsaw v2 is violating the somewhat
hidden contract of the ChainsawViewer interface (this is how you can get the
new Log4UI GUI class to popup if you specify it as an appender in a log4j
configuration file).  Under this circumstance also, an "exit" action is
really an indication that the user is finished with the gui, not necessarily
finished with the VM.

Perhaps we could define an configuration option for Chainsaw which defines
the class name of the Exit action, much the same way you define the Layout
class for an appender.  For your code, we could allow you to just do this
under Chainsaw 2:

...

    LogUI logUI = LogUI.createForInVM();

    logUI.setExitAction(myExitAction);
    logUI.activateViewer();
...

Oh, and have a great vacation!

Thanks again,

Paul

-----Original Message-----
From: Matt Munz [mailto:[EMAIL PROTECTED]
Sent: Friday, 11 July 2003 11:45 AM
To: Log4J Developers List
Subject: RE: Patch: Override ExitAction in Chainsaw


Paul,

  I'm going on vacation and won't be able to look at Chainsaw v2 until next
month :( 

  For my application, the primary feature I'd like to see is the ability to
treat Chainsaw as a software component, rather than just as a standalone
application.  This means making sure that System.exit() is never called, or
is limited to one overrideable point, as indicated in the patch I sent.  

  Just as log4j is embedded within my application, it makes sense to me that
a log4j GUI would be embedded in the GUI for my application.  The more
OO/componetized Chainsaw is, the better for my purposes. This would also be
useful in embedding Chainsaw in other applications like IDEs and server
administration consoles.  Something as simple as a ChainsawPanel that I
could add to my GUI would be really cool...

  BTW, I just had my first use for NDC -- whoever is responsible for this
design, thanks -- it's always nice to find a generic solution that actually
fits my specific problem...

  - Matt
-----Original Message----- 
From: Paul Smith [mailto:[EMAIL PROTECTED] 
Sent: Thu 7/10/2003 7:17 PM 
To: 'Log4J Developers List' 
Cc: 
Subject: RE: Patch: Override ExitAction in Chainsaw


Hi Matt,

Thanks for the patch, but unfortunately the old Chainsaw is now replaced
with the new version in CVS, so I don't think I can apply this one. I'm not
aware of any discussion of an interim 1.2.9 release, I think we're just
working our way towards 1.3.

I haven't had a chance to do much Chainsaw v2 work lately, but if you are
game, perhaps you could try out the new Chainsaw running in-Vm with your
app?  I think I have some spare mental-bandwidth at the moment to help out
with any issues you have.  You're scenario would be a good test for us, as I
don't think we've done much testing along this line, and I would be really
keen to ensure it works really well.

cheers,

Paul Smith

> -----Original Message-----
> From: Matt Munz [mailto:[EMAIL PROTECTED]
> Sent: Friday, 11 July 2003 8:39 AM
> To: [EMAIL PROTECTED]
> Subject: Patch: Override ExitAction in Chainsaw
>
>
> Hi all,
> 
>   This is probably fixed in the latest Chainsaw, but I
> thought I'd forward this patch against 1.2.8 anyway.  In the
> two attached files, the only change is to allow one to pass
> in the Action object that is registered with close events.  I
> needed this functionality to run Chainsaw in-VM with my application.
> 
>   BTW, will there be a 1.2.9 release including the Chainsaw
> changes, or will I have to wait for 1.3?
> 
>   - Matt
> 
> 
>

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

Reply via email to