Explicitly set __hash__ for various classes

Author: TeamSpen210

src/srctools/bsp.py

@@ -111,6 +111,9 @@ class VERSIONS(Enum):
     DESOLATION_OLD = 42  # Old version.
     VITAMINSOURCE = 43  # Desolation's expanded map format.
 
+    def __hash__(self) -> int:
+        return hash(self.value)
+
     def __eq__(self, other: object) -> bool:
         """Versions are equal to their integer value."""
         return self.value == other

src/srctools/dmx.py

@@ -1042,6 +1042,8 @@ def __repr__(self) -> str:
             value = repr(self._value)
         return f'<{self._typ.name} Attr {self.name!r}: {value}>'
 
+    __hash__ = None  # type: ignore[assignment]
+
     def __eq__(self, other: object) -> builtins.bool:
         if isinstance(other, Attribute):
             return (

src/srctools/fgd.py

@@ -534,6 +534,8 @@ def overrides(self) -> Collection[HelperTypes]:
         """
         return ()
 
+    __hash__ = None  # type: ignore[assignment]
+
     def __eq__(self, other: object) -> bool:
         """Define equality as all attributes matching, and only matching types."""
         if not isinstance(other, Helper):
@@ -998,6 +1000,8 @@ def __init__(self, ent: 'EntityDef', attr_name: str, disp_name: str) -> None:
     def __repr__(self) -> str:
         return f'{self._ent!r}.{self._disp_attr}'
 
+    __hash__ = None  # type: ignore[assignment]
+
     def __eq__(self, other: object) -> bool:
         """We're private, so we should be the only instance for a given Entity."""
         return other is self

src/srctools/keyvalues.py

@@ -774,6 +774,8 @@ def as_array(self, *, conv: Callable[[str], T] = cast(Any, str)) -> Union[List[T
         else:
             return [conv(self._value)]
 
+    __hash__ = None  # type: ignore[assignment]
+
     def __eq__(self, other: Any) -> builtins.bool:
         """Compare two items and determine if they are equal.
 

src/srctools/math.py

@@ -323,7 +323,7 @@ class VecBase:
     """Internal Base class for 3D vectors, implementing common code."""
     __match_args__: Final = ('_x', '_y', '_z')
     __slots__ = ('_x', '_y', '_z')
-    __hash__ = None
+    __hash__ = None  # type: ignore[assignment]
 
     #: This is a dictionary containing complementary axes.
     #: ``INV_AXIS["x", "y"]`` gives ``"z"``, and ``INV_AXIS["y"]`` returns ``("x", "z")``.
@@ -1193,7 +1193,7 @@ def __reduce__(self) -> Tuple[Callable[[float, float, float], 'FrozenVec'], Tupl
         """
         return _mk_fvec, (self._x, self._y, self._z)
 
-    def __hash__(self) -> int:
+    def __hash__(self) -> int:  # type: ignore[override]
         """Hashing a frozen vec is the same as hashing the tuple form."""
         return hash((round(self._x, 6), round(self._y, 6), round(self._z, 6)))
 
@@ -1627,7 +1627,7 @@ def _from_raw(
         self._ca, self._cb, self._cc = ca, cb, cc
         return self
 
-    __hash__ = None
+    __hash__ = None  # type: ignore[assignment]
 
     def __eq__(self, other: object) -> bool:
         if isinstance(other, MatrixBase):
@@ -2219,7 +2219,7 @@ class AngleBase:
     # Use the private attrs for matching also, we only hook assignment in the mutable one.
     __match_args__ = ('_pitch', '_yaw', '_roll')
     __slots__ = ('_pitch', '_yaw', '_roll')
-    __hash__ = None
+    __hash__ = None  # type: ignore
 
     _pitch: float
     _yaw: float
@@ -2624,7 +2624,7 @@ def __reduce__(self) -> Tuple[Callable[[float, float, float], 'FrozenAngle'], Tu
         """
         return _mk_fang, (self._pitch, self._yaw, self._roll)
 
-    def __hash__(self) -> int:
+    def __hash__(self) -> int:  # type: ignore[override]
         """Hashing a frozen angle is the same as hashing the tuple form."""
         return hash((round(self._pitch, 6), round(self._yaw, 6), round(self._roll, 6)))
 

src/srctools/tokenizer.py

@@ -59,7 +59,8 @@ def __init__(
     def __repr__(self) -> str:
         return f'TokenSyntaxError({self.mess!r}, {self.file!r}, {self.line_num!r})'
 
-    __hash__ = None  # This is mutable
+    # This is mutable.
+    __hash__ = None  # type: ignore[assignment]
 
     def __eq__(self, other: object) -> bool:
         if isinstance(other, TokenSyntaxError):