@@ -1,6 +1,8 @@
# SPDX-License-Identifier: LGPL-2.1-or-later
# SPDX-FileCopyrightText: 2022 Bartosz Golaszewski <brgl@bgdev.pl>
+from __future__ import annotations
+
from collections import Counter
from errno import ENOENT
from typing import TYPE_CHECKING, Optional, Union
@@ -15,6 +17,7 @@ from .line_settings import LineSettings, _line_settings_to_ext
if TYPE_CHECKING:
from collections.abc import Iterable
from datetime import timedelta
+ from types import TracebackType
from .chip_info import ChipInfo
from .info_event import InfoEvent
@@ -70,14 +73,19 @@ class Chip:
"""
return True if self._chip else False
- def __enter__(self):
+ def __enter__(self) -> Chip:
"""
Controlled execution enter callback.
"""
self._check_closed()
return self
- def __exit__(self, exc_type, exc_value, traceback) -> None:
+ def __exit__(
+ self,
+ exc_type: Optional[type[BaseException]],
+ exc_value: Optional[BaseException],
+ traceback: Optional[TracebackType],
+ ) -> None:
"""
Controlled execution exit callback.
"""
@@ -17,7 +17,7 @@ class ChipInfo:
label: str
num_lines: int
- def __str__(self):
+ def __str__(self) -> str:
return '<ChipInfo name="{}" label="{}" num_lines={}>'.format(
self.name, self.label, self.num_lines
)
@@ -39,7 +39,7 @@ class EdgeEvent:
object.__setattr__(self, "global_seqno", global_seqno)
object.__setattr__(self, "line_seqno", line_seqno)
- def __str__(self):
+ def __str__(self) -> str:
return "<EdgeEvent type={} timestamp_ns={} line_offset={} global_seqno={} line_seqno={}>".format(
self.event_type,
self.timestamp_ns,
@@ -9,7 +9,7 @@ class ChipClosedError(Exception):
Error raised when an already closed chip is used.
"""
- def __init__(self):
+ def __init__(self) -> None:
super().__init__("I/O operation on closed chip")
@@ -18,5 +18,5 @@ class RequestReleasedError(Exception):
Error raised when a released request is used.
"""
- def __init__(self):
+ def __init__(self) -> None:
super().__init__("GPIO lines have been released")
@@ -30,7 +30,7 @@ class InfoEvent:
object.__setattr__(self, "timestamp_ns", timestamp_ns)
object.__setattr__(self, "line_info", line_info)
- def __str__(self):
+ def __str__(self) -> str:
return "<InfoEvent type={} timestamp_ns={} line_info={}>".format(
self.event_type, self.timestamp_ns, self.line_info
)
@@ -15,7 +15,7 @@ class Value(Enum):
INACTIVE = _ext.VALUE_INACTIVE
ACTIVE = _ext.VALUE_ACTIVE
- def __bool__(self):
+ def __bool__(self) -> bool:
return self == self.ACTIVE
@@ -58,7 +58,7 @@ class LineInfo:
self, "debounce_period", timedelta(microseconds=debounce_period_us)
)
- def __str__(self):
+ def __str__(self) -> str:
return '<LineInfo offset={} name="{}" used={} consumer="{}" direction={} active_low={} bias={} drive={} edge_detection={} event_clock={} debounced={} debounce_period={}>'.format(
self.offset,
self.name,
@@ -1,6 +1,8 @@
# SPDX-License-Identifier: LGPL-2.1-or-later
# SPDX-FileCopyrightText: 2022 Bartosz Golaszewski <brgl@bgdev.pl>
+from __future__ import annotations
+
from typing import TYPE_CHECKING, Optional, Union
from . import _ext
@@ -11,6 +13,7 @@ from .line_settings import LineSettings, _line_settings_to_ext
if TYPE_CHECKING:
from collections.abc import Iterable
from datetime import timedelta
+ from types import TracebackType
from .edge_event import EdgeEvent
from .line import Value
@@ -42,14 +45,19 @@ class LineRequest:
"""
return True if self._req else False
- def __enter__(self):
+ def __enter__(self) -> LineRequest:
"""
Controlled execution enter callback.
"""
self._check_released()
return self
- def __exit__(self, exc_type, exc_value, traceback):
+ def __exit__(
+ self,
+ exc_type: Optional[type[BaseException]],
+ exc_value: Optional[BaseException],
+ traceback: Optional[TracebackType],
+ ) -> None:
"""
Controlled execution exit callback.
"""
@@ -81,7 +89,7 @@ class LineRequest:
"""
return self.get_values([line])[0]
- def _check_line_name(self, line):
+ def _check_line_name(self, line: Union[int, str]) -> bool:
if isinstance(line, str):
if line not in self._name_map:
raise ValueError("unknown line name: {}".format(line))
@@ -216,7 +224,7 @@ class LineRequest:
return self._req.read_edge_events(max_events)
- def __str__(self):
+ def __str__(self) -> str:
"""
Return a user-friendly, human-readable description of this request.
"""
@@ -27,7 +27,7 @@ class LineSettings:
# __repr__ generated by @dataclass uses repr for enum members resulting in
# an unusable representation as those are of the form: <NAME: $value>
- def __repr__(self):
+ def __repr__(self) -> str:
return "gpiod.LineSettings(direction=gpiod.line.{}, edge_detection=gpiod.line.{}, bias=gpiod.line.{}, drive=gpiod.line.{}, active_low={}, debounce_period={}, event_clock=gpiod.line.{}, output_value=gpiod.line.{})".format(
str(self.direction),
str(self.edge_detection),
@@ -39,7 +39,7 @@ class LineSettings:
str(self.output_value),
)
- def __str__(self):
+ def __str__(self) -> str:
return "<LineSettings direction={} edge_detection={} bias={} drive={} active_low={} debounce_period={} event_clock={} output_value={}>".format(
self.direction,
self.edge_detection,
Add type hints for all method arguments and return values. Signed-off-by: Vincent Fazio <vfazio@xes-inc.com> --- bindings/python/gpiod/chip.py | 12 ++++++++++-- bindings/python/gpiod/chip_info.py | 2 +- bindings/python/gpiod/edge_event.py | 2 +- bindings/python/gpiod/exception.py | 4 ++-- bindings/python/gpiod/info_event.py | 2 +- bindings/python/gpiod/line.py | 2 +- bindings/python/gpiod/line_info.py | 2 +- bindings/python/gpiod/line_request.py | 16 ++++++++++++---- bindings/python/gpiod/line_settings.py | 4 ++-- 9 files changed, 31 insertions(+), 15 deletions(-)