# HG changeset patch
# User Nicolas Dumazet <nicdumz.commits@gmail.com>
# Date 1279027801 -32400
# Node ID 134eb1c97e94beb833de5822f2a5f74c4e6e2863
# Parent  db9d162337871e3110f3c45dd994db3d28614ba0
mq: qrepo.add(mq.added) inside save_dirty inside of doing it manually

This should make the MQ API more transparent: callers only have to
call save_dirty, and no mq.added magic or knowledge is required.

diff -r db9d16233787 -r 134eb1c97e94 hgext/mq.py
--- a/hgext/mq.py	Tue Jul 06 22:22:18 2010 -0300
+++ b/hgext/mq.py	Tue Jul 13 22:30:01 2010 +0900
@@ -476,6 +476,11 @@
             write_list(self.full_series, self.series_path)
         if self.guards_dirty:
             write_list(self.active_guards, self.guards_path)
+        if self.added:
+            qrepo = self.qrepo()
+            if qrepo:
+                qrepo[None].add(self.added)
+            self.added = []
 
     def removeundo(self, repo):
         undo = repo.sjoin('undo')
@@ -1809,10 +1814,6 @@
               git=opts['git'])
     finally:
         q.save_dirty()
-        qrepo = q.qrepo()
-        if qrepo:
-            qrepo[None].add(q.added)
-        q.added = []
 
     if opts.get('push') and not opts.get('rev'):
         return q.push(repo, None)
diff -r db9d16233787 -r 134eb1c97e94 hgext/rebase.py
--- a/hgext/rebase.py	Tue Jul 06 22:22:18 2010 -0300
+++ b/hgext/rebase.py	Tue Jul 13 22:30:01 2010 +0900
@@ -366,10 +366,6 @@
                 mq.qimport(repo, (), patchname=name, git=isgit,
                                 rev=[str(state[rev])])
         mq.save_dirty()
-        qrepo = mq.qrepo()
-        if qrepo:
-            qrepo[None].add(mq.added)
-        mq.added = []
 
 def storestatus(repo, originalwd, target, state, collapse, keep, keepbranches,
                                                                 external):