Commits

Adrian Buehlmann committed 807eac0

commit: make sure finish() is in scope on exception

  • Participants
  • Parent commits 0162d25
  • Branches stable

Comments (0)

Files changed (1)

tortoisehg/hgtk/commit.py

                 gdialog.Prompt(_('Commit'),
                         _('Unable to create ') + backupdir, self).run()
                 return
+
+        def finish():
+            os.chdir(cwd)
+            # restore backup files
+            try:
+                for realname, tmpname in backups.iteritems():
+                    util.copyfile(tmpname, repo.wjoin(realname))
+                    os.unlink(tmpname)
+                os.rmdir(backupdir)
+            except OSError:
+                pass
+            callback()
+
         try:
             # backup continues
             allchunks = []
                                 _('Unable to apply patch'), self).run()
                         return
 
-            def finish():
-                os.chdir(cwd)
-                # restore backup files
-                try:
-                    for realname, tmpname in backups.iteritems():
-                        util.copyfile(tmpname, repo.wjoin(realname))
-                        os.unlink(tmpname)
-                    os.rmdir(backupdir)
-                except OSError:
-                    pass
-                callback()
-
             # 4. We prepared working directory according to filtered patch.
             #    Now is the time to delegate the job to commit/qrefresh
             #    or the like!