# HG changeset patch
# User Georges Racinet <georges.racinet@octobus.net>
# Date 1619271999 -7200
# Node ID 42eb8b7881b8eb38f0d8a8ccd6aef03b7914c332
# Parent  77e73827a02db25dc675b0afe552f24c807c741d
repoview: style change in newtype() cache handling

This way of writing it does not change the logic at all,
but is more fit for the change we want to make in the
next changeset.

If anything, that's one dict lookup less in the hot path,
but that should be non measurable.

diff -r 77e73827a02d -r 42eb8b7881b8 mercurial/repoview.py
--- a/mercurial/repoview.py	Thu Apr 22 02:57:30 2021 +0200
+++ b/mercurial/repoview.py	Sat Apr 24 15:46:39 2021 +0200
@@ -472,10 +472,12 @@
 
 def newtype(base):
     """Create a new type with the repoview mixin and the given base class"""
-    if base not in _filteredrepotypes:
+    cls = _filteredrepotypes.get(base)
+    if cls is not None:
+        return cls
 
-        class filteredrepo(repoview, base):
-            pass
+    class filteredrepo(repoview, base):
+        pass
 
-        _filteredrepotypes[base] = filteredrepo
-    return _filteredrepotypes[base]
+    _filteredrepotypes[base] = filteredrepo
+    return filteredrepo