From c76b25a17ac49f5ed4827fa7df3a0a3679273ee3 Mon Sep 17 00:00:00 2001 From: itsmattkc Date: Thu, 23 Jul 2020 01:29:04 +1000 Subject: [PATCH] timeline: tools don't need to call move in the release function I had thought that the cursor may still have moved between the last mouseMoveEvent and the mouseReleaseEvent, but that turns out to be false making it completely unnecessary to call move functions in the release function. --- app/widget/timelinewidget/tool/add.cpp | 2 -- app/widget/timelinewidget/tool/transition.cpp | 2 -- 2 files changed, 4 deletions(-) diff --git a/app/widget/timelinewidget/tool/add.cpp b/app/widget/timelinewidget/tool/add.cpp index 58b4e44de..f488bbfca 100644 --- a/app/widget/timelinewidget/tool/add.cpp +++ b/app/widget/timelinewidget/tool/add.cpp @@ -89,8 +89,6 @@ void TimelineWidget::AddTool::MouseMove(TimelineViewMouseEvent *event) void TimelineWidget::AddTool::MouseRelease(TimelineViewMouseEvent *event) { - MouseMove(event); - const TrackReference& track = ghost_->Track(); if (ghost_) { diff --git a/app/widget/timelinewidget/tool/transition.cpp b/app/widget/timelinewidget/tool/transition.cpp index 3b8f0ae40..d3f4d6a74 100644 --- a/app/widget/timelinewidget/tool/transition.cpp +++ b/app/widget/timelinewidget/tool/transition.cpp @@ -106,8 +106,6 @@ void TimelineWidget::TransitionTool::MouseMove(TimelineViewMouseEvent *event) void TimelineWidget::TransitionTool::MouseRelease(TimelineViewMouseEvent *event) { - MouseMove(event); - const TrackReference& track = ghost_->Track(); if (ghost_) {