Skip to content

Commit dbbb8d9

Browse files
committed
Make perf critical places use ignore over cast
1 parent c9893c2 commit dbbb8d9

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

eth/db/diff.py

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
MutableMapping,
44
)
55
from typing import (
6-
cast,
76
Dict,
87
Iterable,
98
Union,
@@ -69,16 +68,16 @@ class DBDiffTracker(ABC_Mutable_Mapping):
6968
def __init__(self) -> None:
7069
self._changes = {} # type: Dict[bytes, Union[bytes, MissingReason]]
7170

72-
def __contains__(self, key: bytes) -> bool: # type: ignore # Breaks LSP
71+
def __contains__(self, key: bytes) -> bool: # type: ignore # Breaks LSP
7372
result = self._changes.get(key, NEVER_INSERTED)
7473
return result not in (DELETED, NEVER_INSERTED)
7574

7675
def __getitem__(self, key: bytes) -> bytes:
7776
result = self._changes.get(key, NEVER_INSERTED)
7877
if result in (DELETED, NEVER_INSERTED):
79-
raise DiffMissingError(key, cast(MissingReason, result))
78+
raise DiffMissingError(key, result) # type: ignore # ignore over cast for perf reasons
8079
else:
81-
return cast(bytes, result)
80+
return result # type: ignore # ignore over cast for perf reasons
8281

8382
def __setitem__(self, key: bytes, value: Union[bytes, MissingReason]) -> None:
8483
self._changes[key] = value
@@ -119,9 +118,9 @@ def __init__(self, changes: Dict[bytes, Union[bytes, MissingReason]] = None) ->
119118
def __getitem__(self, key: bytes) -> bytes:
120119
result = self._changes.get(key, NEVER_INSERTED)
121120
if result in (DELETED, NEVER_INSERTED):
122-
raise DiffMissingError(key, cast(MissingReason, result))
121+
raise DiffMissingError(key, result) # type: ignore # ignore over cast for perf reasons
123122
else:
124-
return cast(bytes, result)
123+
return result # type: ignore # ignore over cast for perf reasons
125124

126125
def __iter__(self) -> None:
127126
raise NotImplementedError(
@@ -151,7 +150,7 @@ def apply_to(self,
151150
else:
152151
pass
153152
else:
154-
db[key] = cast(bytes, value)
153+
db[key] = value # type: ignore # ignore over cast for perf reasons
155154

156155
@classmethod
157156
def join(cls, diffs: Iterable['DBDiff']) -> 'DBDiff':

0 commit comments

Comments
 (0)