Update a few type annotations.

Arguments to functions and methods in the gdb module have mostly been
declared as positional-only arguments up to this point. The idea behind
doing so is to lessen the likelihood of a compatibility issue in the
gdbmongo package if a newer version of GDB chooses to use a different
name as a keyword argument.

Author: visemet

gdbmongo/abseil_printers.py

@@ -123,7 +123,7 @@ class AbslHashMapPrinterBase(AbslPrinterProtocol):
     # pylint: disable=missing-function-docstring
     """Pretty-printer base class for absl::node_hash_map<K, V> and absl::flat_hash_map<K, V>."""
 
-    def __init__(self, val: gdb.Value) -> None:
+    def __init__(self, val: gdb.Value, /) -> None:
         self.key_type = val.type.template_argument(0)
         self.value_type = val.type.template_argument(1)
         self.size = int(val["size_"])

gdbmongo/stdlib_printers.py

@@ -29,11 +29,11 @@
 
     class MyPrinter:
 
-        def __init__(self, val):
+        def __init__(self, val: gdb.Value, /) -> None:
             self.val = val
             self.cursor = val["_cursor"]
 
-        def to_string(self):
+        def to_string(self) -> str:
             # The class is aliased here as a local variable for some brevity.
             UniquePointerPrinter = gdbmongo.stdlib_printers.UniquePointerPrinter
             cursor = UniquePointerPrinter("std::unique_ptr", self.cursor).pointer.dereference()

gdbmongo/stdlib_xmethods.py

@@ -29,11 +29,11 @@
 
     class MyPrinter:
 
-        def __init__(self, val):
+        def __init__(self, val: gdb.Value, /) -> None:
             self.val = val
             self.cursor = val["_cursor"]
 
-        def to_string(self):
+        def to_string(self) -> str:
             xmethod_worker = stdlib_xmethods.UniquePtrMethodsMatcher().match(
                 self.cursor.type, "operator*")
 

stubs/gdb/printing.pyi

@@ -30,9 +30,7 @@ from gdb._value import Value
 class _SupportsDisplayHint(typing.Protocol):
 
     @abc.abstractmethod
-    def display_hint(
-            self
-    ) -> typing.Literal["string"] | typing.Literal["array"] | typing.Literal["map"] | None:
+    def display_hint(self) -> typing.Literal["string", "array", "map", None]:
         raise NotImplementedError
 
 
@@ -77,7 +75,7 @@ class SubPrettyPrinter:
 
 class RegexpCollectionPrettyPrinter(PrettyPrinter):
 
-    def __init__(self, name: str) -> None:
+    def __init__(self, name: str, /) -> None:
         ...
 
     def add_printer(self, name: str, regexp: str,
@@ -93,7 +91,7 @@ class FlagEnumerationPrinter(PrettyPrinter):
 
 
 def register_pretty_printer(obj: Objfile | Progspace | None,
-                            printer: typing.Callable[[Value], _SupportsToString
-                                                     | _SupportsChildren | None],
-                            replace: bool = False) -> None:
+                            printer: typing.Callable[[Value],
+                                                     _SupportsToString | _SupportsChildren | None],
+                            /, *, replace: bool = False) -> None:
     ...