Kiln » TortoiseHg » TortoiseHg
Clone URL:  
Pushed to one repository · View In Graph Contained in 0.9, 0.9.1, and 0.9.1.1

visdiff: connect response and should_live notifications

Fixes leaking of temp files and properly saves the settings
Fixes #631

Changeset e5781f702998

Parent fb9be2b7f13a

by Steve Borho

Changes to one file · Browse files at e5781f702998 Showing diff from parent fb9be2b7f13a Diff from another changeset...

 
184
185
186
 
187
188
189
 
207
208
209
210
 
211
212
213
 
 
 
 
 
 
 
214
215
216
 
184
185
186
187
188
189
190
 
208
209
210
 
211
212
213
214
215
216
217
218
219
220
221
222
223
224
@@ -184,6 +184,7 @@
    tmproot = tempfile.mkdtemp(prefix='extdiff.')   self.connect('destroy', self.delete_tmproot, tmproot) + self.connect('response', self.delete_tmproot_resp, tmproot)   dir2 = ''   dir2root = ''   # Always make a copy of node1 @@ -207,10 +208,17 @@
  if len(model) == 1 and self.singlecheck.get_active():   self.launch(*model[0])   - def delete_tmproot(self, window, tmproot): + def should_live(self, widget=None, event=None):   vsettings = settings.Settings('visdiff')   vsettings.set_value('launchsingle', self.singlecheck.get_active())   vsettings.write() + return False + + def delete_tmproot_resp(self, window, resp, tmproot): + self.delete_tmproot(window, tmproot) + + def delete_tmproot(self, window, tmproot): + self.should_live()   while True:   try:   shutil.rmtree(tmproot)