# HG changeset patch
# User Sean Farley <sean@farley.io>
# Date 1494536388 25200
# Node ID bf728e72a21901b0ae9170958ade16893c6938e3
# Parent  c0b16b801687be26d14cabe501678e24b2a3c117
context: move dirty() to committablectx

This is a pedantic move. It should be an error if dirty() is called on a
read-only context. Based on Mads Kiilerix's and my work at the sprint.

diff -r c0b16b801687 -r bf728e72a219 mercurial/context.py
--- a/mercurial/context.py	Thu May 11 13:51:10 2017 -0700
+++ b/mercurial/context.py	Thu May 11 13:59:48 2017 -0700
@@ -323,9 +323,6 @@
     def hasdir(self, dir):
         return self._manifest.hasdir(dir)
 
-    def dirty(self, missing=False, merge=True, branch=True):
-        return False
-
     def status(self, other=None, match=None, listignored=False,
                listclean=False, listunknown=False, listsubrepos=False):
         """return status of files between two nodes or node and working
@@ -1569,6 +1566,9 @@
         # from immediately doing so for subsequent changing files
         self._repo.dirstate.write(self._repo.currenttransaction())
 
+    def dirty(self, missing=False, merge=True, branch=True):
+        return False
+
 class workingctx(committablectx):
     """A workingctx object makes access to data related to
     the current working directory convenient.