config: use a new `alter` method in `fixconfig`
authorPierre-Yves David <pierre-yves.david@octobus.net>
Sat, 30 Jan 2021 00:32:42 +0100
changeset 46620 7621ab4005bf
parent 46619 f2fc34e88238
child 46621 d3df397e7a59
config: use a new `alter` method in `fixconfig` The `set` function is doing various work related to metadata (eg: the source, later the level). However the `fixconfig` call only updating some values according to standard processing, not changing any of the related metadata. So we introduce a new method and use it there. Differential Revision: https://phab.mercurial-scm.org/D9924
mercurial/config.py
mercurial/ui.py
--- a/mercurial/config.py	Mon Mar 01 14:44:29 2021 -0800
+++ b/mercurial/config.py	Sat Jan 30 00:32:42 2021 +0100
@@ -108,6 +108,16 @@
             self._source = self._source.preparewrite()
             self._source[(section, item)] = source
 
+    def alter(self, section, key, new_value):
+        """alter a value without altering its source or level
+
+        This method is meant to be used by `ui.fixconfig` only."""
+        item = self._data[section][key]
+        size = len(item)
+        new_item = (new_value,) + item[1:]
+        assert len(new_item) == size
+        self._data[section][key] = new_item
+
     def restore(self, data):
         """restore data returned by self.backup"""
         self._source = self._source.preparewrite()
--- a/mercurial/ui.py	Mon Mar 01 14:44:29 2021 -0800
+++ b/mercurial/ui.py	Sat Jan 30 00:32:42 2021 +0100
@@ -554,7 +554,7 @@
                     p = util.expandpath(p)
                     if not util.hasscheme(p) and not os.path.isabs(p):
                         p = os.path.normpath(os.path.join(root, p))
-                    c.set(b"paths", n, p)
+                    c.alter(b"paths", n, p)
 
         if section in (None, b'ui'):
             # update ui options