Jeremy and Ceki,

Looking at the code, the close() method seems to do what I thought it should
do.  Jeremy, could you describe your setup, and what would (or should) be
triggering a closing of the appender?  

Is it because the VM is exiting without a proper Log4j shutdown and relying
on the finalize being called by the vm (which frankly I have less and less
faith in finalize in VM's, they should rename the method
crossYourFingersFinalize()).

This does make me ask myself whether Log4j could add itself as a shutdown
hook, but I think that is 1.3 specific isn't it? If so, perhaps this would
be a candidate for some sort of dynamic/reflective magic on detection of 1.3
to use this feature for tidying up since it's pretty useful and avoid a
compile time dependancy on 1.3

cheers,

Paul Smith

> -----Original Message-----
> From: Ceki Gülcü [mailto:[EMAIL PROTECTED]
> Sent: Tuesday, 22 July 2003 8:49 AM
> To: Log4J Developers List
> Subject: Re: HtmlLayout not closing the table
> 
> 
> 
> This problem has been mentioned before. I think it is also 
> reported as a 
> bug in bugzilla (http://nagoya.apache.org/bugzilla/) but I 
> could not find 
> the report.
> 
> You might want to file a new report.
> 
> At 04:19 PM 7/21/2003 -0400, you wrote:
> 
> >I've downloaded the latest source from apache to enhance the 
> HtmlLayout to 
> >display the time field as the current date instead of the 
> time elapsed 
> >since the LayoutManager had been loaded.  When inside the code and 
> >testing, I found that the getFooter method was never being 
> called, thus 
> >never closing the body, table, html tags.  Has anyone come across 
> >this?  Does someone have a good idea on how to resolve this? 
>  I'm new to 
> >this code, so I've been hunting around some and did find that the 
> >finalize() method of the WriterAppender is suppose to call 
> getFooter(), 
> >but this method is never called.
> >
> >_________________
> >Jeremy Nix
> >Senior Application Developer
> >Southwest Financial Ltd.
> >[EMAIL PROTECTED]
> >(513) 621-6699 ext 1158
> 
> --
> Ceki  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]
> 

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

Reply via email to