Bug 364305 - git master: moving a single transition using the mouse crashes Kdenlive immediately of mouse release
Summary: git master: moving a single transition using the mouse crashes Kdenlive immed...
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (other bugs)
Version First Reported In: unspecified
Platform: Compiled Sources Linux
: NOR major
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-14 09:06 UTC by Wegwerf
Modified: 2016-06-14 20:16 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Wegwerf 2016-06-14 09:06:51 UTC
With the recent refactoring in git master there is a nasty bug that causes Kdenlive to crash immediately after trying to move a transition in the timeline. The crash is in TransitionHandler::moveTransition().

Reproducible: Always

Steps to Reproduce:
1. Start new project.
2. Add a (color) clip to your project and timeline.
3. Add a transition to the clip in the timeline.
4. Drag (move) transition in timeline using the mouse.

Actual Results:  
Kdenlive crashes in TransitionHandler::moveTransition() in reaction to MoveManager::mouseRelease().

Expected Results:  
Transition moves correctly to new position.
Comment 1 Wegwerf 2016-06-14 09:07:25 UTC
By the way: moving a group containing transitions does not crash Kdenlive. (phew!)
Comment 2 Wegwerf 2016-06-14 09:24:24 UTC
Being a git master tester surely drives creative thinking: I worked around by copying the transition, then marking both transitions so I could move them now without crash. Finally, I deleted the second transition that isn't needed anymore...
Comment 3 Jean-Baptiste Mardelle 2016-06-14 20:02:12 UTC
Git commit 49673eec7e62cc02312501c344e2105b6916c76c by Jean-Baptiste Mardelle.
Committed on 14/06/2016 at 20:01.
Pushed by mardelle into branch 'master'.

Fix crash on transition move

M  +11   -7    src/timeline/customtrackview.cpp
M  +1    -0    src/timeline/customtrackview.h
M  +1    -0    src/timeline/timeline.cpp

http://commits.kde.org/kdenlive/49673eec7e62cc02312501c344e2105b6916c76c
Comment 4 Wegwerf 2016-06-14 20:16:48 UTC
Great! Crashes are fixed, thank you very much!