[Kdenlive-devel] problems with latest svn/git

[Mikko Rapeli]

On Tue, Jun 22, 2010 at 10:53:40PM +0300, Mikko Rapeli wrote:
> Is this similar to not being able to move a clip on timeline after adjusting
> other clips start and end points before that clip?
> 
> I checked again to git master and this problem was the first problem I hit
> with my project.

Moved start and end positions of two clips on same track prior to a third
clip. Third clip can no longer be moved due some error. git bisect and patch
splitting points to the patch below.

On Tue, Jun 22, 2010 at 10:17:09PM +0300, Mikko Rapeli wrote:
> diff --git a/src/renderer.cpp b/src/renderer.cpp
> index ad5793d..247e913 100644
> --- a/src/renderer.cpp
> +++ b/src/renderer.cpp
> @@ -2750,17 +2750,15 @@ bool Render::mltResizeClipStart(ItemInfo info, GenTime diff)
>      }
>      mlt_service_lock(service.get_service());
>      int clipIndex = trackPlaylist.get_clip_index_at(info.startPos.frames(m_fps));
> -    Mlt::Producer *clip = trackPlaylist.get_clip(clipIndex);
> -    if (clip == NULL) {
> +    if (trackPlaylist.is_blank(clipIndex)) {
>          kDebug() << "////////  ERROR RSIZING NULL CLIP!!!!!!!!!!!";
>          mlt_service_unlock(service.get_service());
>          return false;
>      }
> -    int previousStart = clip->get_in();
> -    delete clip;
> +    int previousStart = trackPlaylist.clip_start(clipIndex);
>      int previousDuration = trackPlaylist.clip_length(clipIndex) - 1;
>      m_isBlocked = true;
> -    kDebug() << "RESIZE, old start: " << previousStart << ", PREV DUR: " << previousDuration << ", DIFF: " << moveFrame;
> +    kDebug() << "RESIZE, old start: " << previousStart + moveFrame<<", "<<previousStart + previousDuration;
>      trackPlaylist.resize_clip(clipIndex, previousStart + moveFrame, previousStart + previousDuration);
>      if (moveFrame > 0) trackPlaylist.insert_blank(clipIndex, moveFrame - 1);
>      else {