Skip to content

Commit ecc904a

Browse files
authored
Merge pull request #91 from chennes/removeUnusedImport
Simplify __del__ methods and add explicit option
2 parents a3dcdaf + 1f2ffb4 commit ecc904a

File tree

2 files changed

+16
-8
lines changed

2 files changed

+16
-8
lines changed

addonmanager_freecad_interface.py

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -264,14 +264,19 @@ def __init__(self):
264264
def __del__(self):
265265
self.reference_count -= 1
266266
if not FreeCAD and self.reference_count <= 0:
267-
paths = [self.data_dir, self.mod_dir, self.cache_dir, self.macro_dir, self.mod_dir]
268-
for path in paths:
269-
if os.path.isdir(path):
270-
os.rmdir(path)
271-
self.data_dir = None
272-
self.mod_dir = None
273-
self.cache_dir = None
274-
self.macro_dir = None
267+
self._delete_paths()
268+
269+
def _delete_paths(self):
270+
if FreeCAD:
271+
return
272+
paths = [self.data_dir, self.mod_dir, self.cache_dir, self.macro_dir, self.mod_dir]
273+
for path in paths:
274+
if os.path.isdir(path):
275+
os.rmdir(path)
276+
self.data_dir = None
277+
self.mod_dir = None
278+
self.cache_dir = None
279+
self.macro_dir = None
275280

276281

277282
class Preferences:

addonmanager_installer_gui.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -536,6 +536,9 @@ def __init__(self, addon):
536536
self.macro_dir = fci.DataPaths().macro_dir
537537

538538
def __del__(self):
539+
quit()
540+
541+
def quit(self):
539542
if self.worker_thread and hasattr(self.worker_thread, "quit"):
540543
self.worker_thread.quit()
541544
self.worker_thread.wait(500)

0 commit comments

Comments
 (0)