Skip to content

Commit 0b4a160

Browse files
authored
Use slots=True on all dataclasses. (pelme#112)
No big deal but nicer to be strict/potentially save some memory.
1 parent 97f3c4b commit 0b4a160

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

htpy/__init__.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ def _attrs_string(attrs: Mapping[str, Attribute]) -> str:
129129
P = t.ParamSpec("P")
130130

131131

132-
@dataclasses.dataclass(frozen=True)
132+
@dataclasses.dataclass(frozen=True, slots=True)
133133
class ContextProvider(t.Generic[T]):
134134
context: Context[T]
135135
value: T
@@ -154,7 +154,7 @@ def encode(self, encoding: str = "utf-8", errors: str = "strict") -> bytes:
154154
return str(self).encode(encoding, errors)
155155

156156

157-
@dataclasses.dataclass(frozen=True)
157+
@dataclasses.dataclass(frozen=True, slots=True)
158158
class ContextConsumer(t.Generic[T]):
159159
context: Context[T]
160160
debug_name: str
@@ -183,7 +183,7 @@ class _NO_DEFAULT:
183183
pass
184184

185185

186-
@dataclasses.dataclass(frozen=True)
186+
@dataclasses.dataclass(frozen=True, slots=True)
187187
class Context(t.Generic[T]):
188188
name: str
189189
_: dataclasses.KW_ONLY

t.py

Whitespace-only changes.

tests/test_context.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
from .conftest import RenderFixture
1111

1212
letter_ctx: Context[t.Literal["a", "b", "c"]] = Context("letter", default="a")
13-
no_default_ctx = Context[str]("no_default")
13+
no_default_ctx: Context[str] = Context("no_default")
1414

1515

1616
@letter_ctx.consumer

0 commit comments

Comments
 (0)