( )⚙ D11967 simplemerge: rewrite `merge_lines()` using `merge_groups()`

This is an archive of the discontinued Mercurial Phabricator instance.

simplemerge: rewrite `merge_lines()` using `merge_groups()`
ClosedPublic

Authored by martinvonz on Jan 6 2022, 3:35 PM.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

martinvonz created this revision.Jan 6 2022, 3:35 PM
martinvonz added a comment.EditedJan 6 2022, 3:38 PM

I refactored it a bit to try to convince pytype. That resulted in D11966. I still don't understand what pytype was thinking. It didn't help to add explicit type annotations (something like Union[Tuple[bytes, List[bytes]], Tuple[bytes, List[bytes], List[bytes], List[bytes]]]).

Alphare accepted this revision.Jan 7 2022, 4:52 AM
Alphare added a subscriber: Alphare.

Superseeds D11904

This revision is now accepted and ready to land.Jan 7 2022, 4:52 AM