Patchwork D10980: dirstate-entry: add a `size` property

login
register
mail settings
Submitter phabricator
Date July 4, 2021, 9:55 p.m.
Message ID <differential-rev-PHID-DREV-7nigx7c7nq2o34wktus5-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/49293/
State Superseded
Headers show

Comments

phabricator - July 4, 2021, 9:55 p.m.
marmoute created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  This is clearer than "tuple" indexing.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D10980

AFFECTED FILES
  mercurial/cext/parsers.c
  mercurial/dirstate.py
  mercurial/pure/parsers.py

CHANGE DETAILS




To: marmoute, #hg-reviewers
Cc: mercurial-patches, mercurial-devel

Patch

diff --git a/mercurial/pure/parsers.py b/mercurial/pure/parsers.py
--- a/mercurial/pure/parsers.py
+++ b/mercurial/pure/parsers.py
@@ -76,6 +76,10 @@ 
         return self._mode
 
     @property
+    def size(self):
+        return self._size
+
+    @property
     def state(self):
         """
         States are:
diff --git a/mercurial/dirstate.py b/mercurial/dirstate.py
--- a/mercurial/dirstate.py
+++ b/mercurial/dirstate.py
@@ -1309,7 +1309,7 @@ 
             # tuple members one by one.
             t = dget(fn)
             mode = t.mode
-            size = t[2]
+            size = t.size
             time = t[3]
 
             if not st and t.tracked:
diff --git a/mercurial/cext/parsers.c b/mercurial/cext/parsers.c
--- a/mercurial/cext/parsers.c
+++ b/mercurial/cext/parsers.c
@@ -174,6 +174,11 @@ 
 	return PyInt_FromLong(self->mode);
 };
 
+static PyObject *dirstatetuple_get_size(dirstateTupleObject *self)
+{
+	return PyInt_FromLong(self->size);
+};
+
 static PyObject *dirstatetuple_get_state(dirstateTupleObject *self)
 {
 	return PyBytes_FromStringAndSize(&self->state, 1);
@@ -244,6 +249,7 @@ 
 
 static PyGetSetDef dirstatetuple_getset[] = {
     {"mode", (getter)dirstatetuple_get_mode, NULL, "mode", NULL},
+    {"size", (getter)dirstatetuple_get_size, NULL, "size", NULL},
     {"state", (getter)dirstatetuple_get_state, NULL, "state", NULL},
     {"tracked", (getter)dirstatetuple_get_tracked, NULL, "tracked", NULL},
     {"added", (getter)dirstatetuple_get_added, NULL, "added", NULL},