[comixed-dev] Re: Cleaning up unclosed ZipFile for archivemessagesin CMD window.

  • From: "bareheiny" <dmarc-noreply@xxxxxxxxxxxxx> (Redacted sender "bareheiny" for DMARC)
  • To: comixed-dev@xxxxxxxxxxxxx
  • Date: Sat, 14 Mar 2020 08:23:04 +1300

Ah - so is that a change that was made in the last build or so?

I’m just capturing a log for WebP archives....and that message is occurring a 
lot more.  It looks like once per page in the archive - which is what the log 
looks like as well - a massive number of “loading entry: name=P0015.webp size=“ 
type entries.

It feels like it may be part of trying to find cover images for the import 
page.  I’ll get some logs and a screenshot up on GitHub as part of a bug.

On 14/03/2020, at 02:31, Darryl L. Pierce <mcpierce@xxxxxxxxx> wrote:

Okay, googled the error message and found that it's just some logging
from the Apache Compress library we're using to handle CBZ files.
Instead of using Log4J the Apache code is writing that output to
stderror when it cleans up its own resources.


/**
* Ensures that the close method of this zipfile is called when
* there are no more references to it.
* @see #close()
*/
@Override
protected void finalize() throws Throwable {
 try {
   if (!closed) {
     System.err.println("Cleaning up unclosed ZipFile for archive "
               + archiveName);
     close();
   }
 } finally {
   super.finalize();
 }
}
---8<[snip]---

https://www.codota.com/code/java/methods/org.apache.commons.compress.archivers.zip.ZipFile/close


On Fri, Mar 13, 2020 at 6:47 AM Guy Incognito
<dmarc-noreply@xxxxxxxxxxxxx> wrote:

I’ve managed to trigger the “Cleaning up unclosed...” message in JPG 
archives.



Process seems to be:



Start an import
Navigate to the library page
Select one, or more, comics that are scheduled for processing
Check the CMD window for the message



I’ll need to try with the 0.5RC, and maybe a few older 0.6 development 
versions...I haven’t struck this messaging before, and the workflow I use 
for processing comics hasn’t changed notably.



From: Guy Incognito
Sent: Friday, 13 March 2020 11:43 AM
To: comixed-dev@xxxxxxxxxxxxx
Subject: [comixed-dev] Re: Cleaning up unclosed ZipFile for 
archivemessagesin CMD window.



Here’s one of the messages from the CMD window text:



Cleaning up unclosed ZipFile for archive N:\Comics\Motor Girl\Motor Girl 
Vol.2016 #09 (October, 2017).cbz



I’m seeing the same clean up message multiple times – it may go MG #9, MG 
#10, Another series #3, MG #9 and so on.



I do need to capture logs for importing the same comics in JPG and WebP 
format and compare the logs – I *think* I’m seeing CX behave slightly 
differently for the WebP archives.



Capturing two sets of logs will also let me see if there’s something going 
on with the way I’m processing the comics - as the only different step is 
the conversion to WebP.





From: Darryl L. Pierce
Sent: Friday, 13 March 2020 2:38 AM
To: comixed-dev@xxxxxxxxxxxxx
Subject: [comixed-dev] Re: Cleaning up unclosed ZipFile for archive 
messagesin CMD window.



Can you paste the exact message from the console? I've not seen anything 
like that previously.



On Wed, Mar 11, 2020 at 6:37 PM Guy Incognito <dmarc-noreply@xxxxxxxxxxxxx> 
wrote:

Just wondering what would cause that – not sure if it’s something that’s 
occurred because of the way I process my comics now, or if it’s CX.

It seems to happen when I navigate to the library page of CX while WebP 
archives are being processed (note I’ve installed parts of Visual 
Studio...so the WebP library is working now).



In saying that, I have noticed that cover images aren’t being extracted from 
the WebP archives at the import selection stage (once fully imported, the 
covers display with no issues)...but I’m not raising that until the WebP 
library is working properly, rather than relying on a user having VS 
installed.





--

Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord 
Downie







-- 
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie



Other related posts: