On Friday 17 February 2012 11:23:19 Simon A. Eugster wrote: > On 02/16/2012 09:27 PM, jb wrote:
> > > > 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. > > Thanks a lot! This works now. But I have another problem: > 1. Add d7000.wav on track 1 > 2. Add h4n.wav on track 1 too, to the right of d7000.wav, and set it as > reference > 3. Auto-align d7000.wav, this should give an error imho but does not > 4. Move d7000.wav to track 2 -> undo stack corrupted We did not check if the region was empty in MLT before moving the clip. If I remember correctly, in most places in Kdenlive we first check if the clip can be moved using the GraphicsView coordinates (check that no other clip overlap) and the make the move in MLT playlist without further verification. So in your case, the move was first made in MLT but then failed in Kdenlive (because of overlapping clips). So I guess this is where a refactoring would help, because currently we have the Kdenlive timeline graphics view AND the MLT playlist, and sometimes those 2 can lose syncro in these situations. I think your problem should be fixed by my last commit to the audioAlign branch where I now also check in MLT that the region where we move the clip is empty. regards jb ------------------------------------------------------------------------------ Virtualization & Cloud Management Using Capacity Planning Cloud computing makes use of virtualization - but cloud computing also focuses on allowing computing to be delivered as a service. http://www.accelacomm.com/jaw/sfnl/114/51521223/ _______________________________________________ Kdenlive-devel mailing list Kdenlive-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kdenlive-devel
