qapi: Bury some unused code in class Indentation
authorMarkus Armbruster <armbru@redhat.com>
Wed, 8 Sep 2021 04:54:26 +0000 (06:54 +0200)
committerMarkus Armbruster <armbru@redhat.com>
Wed, 8 Sep 2021 13:30:10 +0000 (15:30 +0200)
.__int__() has never been used.  Drop it.

.decrease() raises ArithmeticError when asked to decrease indentation
level below zero.  Nothing catches it.  It's a programming error.
Dumb down to assert.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20210908045428.2689093-4-armbru@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
scripts/qapi/common.py

index 1d62c27fb7485b5381eba8a8f8bdaa8f5a03c4aa..489273574aeee8ddf6d5aa38c8291f88e1343eef 100644 (file)
@@ -132,9 +132,6 @@ class Indentation:
     def __init__(self, initial: int = 0) -> None:
         self._level = initial
 
-    def __int__(self) -> int:
-        return self._level
-
     def __repr__(self) -> str:
         return "{}({:d})".format(type(self).__name__, self._level)
 
@@ -148,9 +145,7 @@ class Indentation:
 
     def decrease(self, amount: int = 4) -> None:
         """Decrease the indentation level by ``amount``, default 4."""
-        if self._level < amount:
-            raise ArithmeticError(
-                f"Can't remove {amount:d} spaces from {self!r}")
+        assert amount <= self._level
         self._level -= amount