On Sat, 17 Feb 2024 at 11:18, Akihiko Odaki <akihiko.od...@daynix.com> wrote: > > ui/cocoa used to release all mouse buttons when it sees > NSEventTypeLeftMouseUp, NSEventTypeRightMouseUp, or > NSEventTypeOtherMouseUp, but it can instead release specific one > according to the delivered event. > > Signed-off-by: Akihiko Odaki <akihiko.od...@daynix.com> > --- > ui/cocoa.m | 132 > ++++++++++++++++++++++++++----------------------------------- > 1 file changed, 55 insertions(+), 77 deletions(-)
This is a pain to review because it does three things at once: (1) a no-behaviour change refactoring where various cases that previously did 'break' now 'return true' (2) a refactoring so mouse events are handled in a separate method, rather than by setting the mouse_event = true bool and handling this at the bottom of the handleEventLocked method (3) the actual behaviour change that the comment message talks about If this was three patches that each did one of those things, two of them would be trivial to review and the code changes required for 3 would be easy to find rather than buried in with the changes for the first two. thanks -- PMM
