[Kdenlive-devel] Need help with the Move Clip command

[jb]

On Thursday 16 February 2012 21:23:14 jb wrote:
> On Thursday 16 February 2012 21:00:52 jb wrote:
> > On Wednesday 15 February 2012 21:50:23 Simon A. Eugster wrote:
> > > The MoveClipCommand now moves d7000 from 42 s to 0 s instead. Trying to
> > > undo this action previously attempted to move the clip at t=-3 to t=42
> > > again which obviously failed since there never was a clip at t=-3.
> > > 
> > > This is fixed now, so undo tries to move the clip at t=0 back to t=42.
> > > But it fails! [1] WHY ON EARTH! I'm told that the place I want to move
> > > my clip to is already occupied but it is blanker than blank emptyness.
> > > 
> > > Any help appreciated.
> > > thanks
> > > Simon
> > 
> > I am looking at it, and the problem seems to be in the align command. It
> > does not successfully move the clip from 42s to 0s (the clip is moved in
> > the Kdenlive timeline but seems to be deleted from the MLT track
> > playlist...)
> > 
> > I am going to have a closer look at the source to find the problem, will
> > post the results asap.
> 
> The problem was that you tried to move a clip to a negative timecode. MLT
> does not support it and Kdenlive did not correctly detect the failure.
> 
> In case of negative timecode, you should do a 2 step command. For example,
> if you want to move a clip to -3 seconds:
> 
> 1) Resize clip start to crop the first 3 seconds
> 2) Move the clip to 0
> 

Forgot to say that I committed a patch to the AudioAlign branch so that 
Kdenlive now displays an error instead of moving the clip in the first place.

There are probably other placed in Kdenlive where the "playlist.insert_at" 
return value is not checked, will keep that on my todo list.

jb