The class vars in ZipDocument should be instance vars, otherwise they end up accumulating state across multiple ZipDocument.load invocations through rmapy.Client.download calls.
Found this while downloading and dumping to disk some PDFs. Lead to hundreds of .rm files in directories as it accumulates state in self.rm (and other vars!)
subutux
commented
3 years ago
The class vars in
ZipDocumentshould be instance vars, otherwise they end up accumulating state across multipleZipDocument.loadinvocations throughrmapy.Client.downloadcalls.Found this while downloading and dumping to disk some PDFs. Lead to hundreds of
.rmfiles in directories as it accumulates state inself.rm(and other vars!)@subutux for review