This is an archive of the discontinued Mercurial Phabricator instance.

Differential D11645

chistedit: move makeselection() onto state class
ClosedPublic

Authored by martinvonz on Oct 12 2021, 1:05 PM.

Details

Reviewers
durin42
pulkit
Group Reviewers
hg-reviewers
Commits
rHG0f498e03b016: chistedit: move makeselection() onto state class

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

martinvonz created this revision.Oct 12 2021, 1:05 PM
pulkit accepted this revision.Oct 13 2021, 3:08 PM
This revision is now accepted and ready to land.Oct 13 2021, 3:08 PM
martinvonz added a commit: rHG0f498e03b016: chistedit: move makeselection() onto state class.Oct 13 2021, 6:27 PM
Closed by commit rHG0f498e03b016: chistedit: move makeselection() onto state class (authored by martinvonz). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mhgext/histedit.py (11 lines)

Diff 30733

hgext/histedit.py

return self.conflicts return self.conflicts
if other in self.conflicts: if other in self.conflicts:
self.conflicts.remove(other) self.conflicts.remove(other)
return self.conflicts return self.conflicts
# ============ EVENTS =============== # ============ EVENTS ===============
def makeselection(state, pos):
state.selected = pos
def swap(state, oldpos, newpos): def swap(state, oldpos, newpos):
"""Swap two positions and calculate necessary conflicts in """Swap two positions and calculate necessary conflicts in
O(|newpos-oldpos|) time""" O(|newpos-oldpos|) time"""
rules = state.rules rules = state.rules
assert 0 <= oldpos < len(rules) and 0 <= newpos < len(rules) assert 0 <= oldpos < len(rules) and 0 <= newpos < len(rules)
rules[oldpos], rules[newpos] = rules[newpos], rules[oldpos] rules[oldpos], rules[newpos] = rules[newpos], rules[oldpos]
# TODO: swap should not know about histeditrule's internals # TODO: swap should not know about histeditrule's internals
rules[newpos].pos = newpos rules[newpos].pos = newpos
rules[oldpos].pos = oldpos rules[oldpos].pos = oldpos
start = min(oldpos, newpos) start = min(oldpos, newpos)
end = max(oldpos, newpos) end = max(oldpos, newpos)
for r in pycompat.xrange(start, end + 1): for r in pycompat.xrange(start, end + 1):
rules[newpos].checkconflicts(rules[r]) rules[newpos].checkconflicts(rules[r])
rules[oldpos].checkconflicts(rules[r]) rules[oldpos].checkconflicts(rules[r])
if state.selected: if state.selected:
makeselection(state, newpos) state.make_selection(newpos)
def changeaction(state, pos, action): def changeaction(state, pos, action):
"""Change the action state on the given position to the new action""" """Change the action state on the given position to the new action"""
rules = state.rules rules = state.rules
assert 0 <= pos < len(rules) assert 0 <= pos < len(rules)
rules[pos].action = action rules[pos].action = action
if selected is not None or action == b'move-up': if selected is not None or action == b'move-up':
swap(self, oldpos, newpos) swap(self, oldpos, newpos)
elif action == b'next-action': elif action == b'next-action':
cycleaction(self, oldpos, next=True) cycleaction(self, oldpos, next=True)
elif action == b'prev-action': elif action == b'prev-action':
cycleaction(self, oldpos, next=False) cycleaction(self, oldpos, next=False)
elif action == b'select': elif action == b'select':
selected = oldpos if selected is None else None selected = oldpos if selected is None else None
makeselection(self, selected) self.make_selection(selected)
elif action == b'goto' and int(ch) < len(rules) and len(rules) <= 10: elif action == b'goto' and int(ch) < len(rules) and len(rules) <= 10:
newrule = next((r for r in rules if r.origpos == int(ch))) newrule = next((r for r in rules if r.origpos == int(ch)))
self.move_cursor(oldpos, newrule.pos) self.move_cursor(oldpos, newrule.pos)
if selected is not None: if selected is not None:
swap(self, oldpos, newrule.pos) swap(self, oldpos, newrule.pos)
elif action.startswith(b'action-'): elif action.startswith(b'action-'):
changeaction(self, oldpos, action[7:]) changeaction(self, oldpos, action[7:])
elif action == b'showpatch': elif action == b'showpatch':
self.modes[MODE_PATCH][b'line_offset'] = 0 self.modes[MODE_PATCH][b'line_offset'] = 0
def change_mode(self, mode): def change_mode(self, mode):
curmode, _ = self.mode curmode, _ = self.mode
self.mode = (mode, curmode) self.mode = (mode, curmode)
if mode == MODE_PATCH: if mode == MODE_PATCH:
self.modes[MODE_PATCH][b'patchcontents'] = self.patch_contents() self.modes[MODE_PATCH][b'patchcontents'] = self.patch_contents()
def make_selection(self, pos):
self.selected = pos
def _chisteditmain(repo, rules, stdscr): def _chisteditmain(repo, rules, stdscr):
try: try:
curses.use_default_colors() curses.use_default_colors()
except curses.error: except curses.error:
pass pass
# initialize color pattern # initialize color pattern