Mercurial > public > mercurial-scm > hg-stable
diff mercurial/revlog.py @ 52497:4ef6dbc27a99
revlog: subclass the new `repository.iverifyproblem` Protocol class
This is the same transformation as 3a90a6fd710d did for dirstate, but the
CamelCase naming was already cleaned up here.
We shouldn't have to explicitly subclass, but I'm doing so to test the interplay
of regular attributes and the `attrs` class. Also, PyCharm has a nifty feature
that puts a jump point in the gutter to navigate back and forth between the base
class and subclasses (and override functions and base class functions) when
there's an explicit subclassing. Additionally, PyCharm will immediately flag
signature mismatches without a 40m pytype run.
| author | Matt Harbison <matt_harbison@yahoo.com> |
|---|---|
| date | Tue, 22 Oct 2024 17:18:26 -0400 |
| parents | 4eec920bbb37 |
| children | 43d2a2f66434 |
line wrap: on
line diff
--- a/mercurial/revlog.py Tue Oct 22 17:07:23 2024 -0400 +++ b/mercurial/revlog.py Tue Oct 22 17:18:26 2024 -0400 @@ -204,7 +204,7 @@ @attr.s(frozen=True) -class revlogproblem: # (repository.iverifyproblem) +class revlogproblem(repository.iverifyproblem): warning = attr.ib(default=None, type=Optional[bytes]) error = attr.ib(default=None, type=Optional[bytes]) node = attr.ib(default=None, type=Optional[bytes]) @@ -4021,7 +4021,7 @@ else: rewrite.v2_censor(self, tr, censor_nodes, tombstone) - def verifyintegrity(self, state) -> Iterable[revlogproblem]: + def verifyintegrity(self, state) -> Iterable[repository.iverifyproblem]: """Verifies the integrity of the revlog. Yields ``revlogproblem`` instances describing problems that are