From: Dan Dennedy <dan@de...> - 2007-08-02 19:02:10
On Thursday 02 August 2007 9:19:07 am Ryan Hodge wrote:
> Hey Dan,
> I'm a member of the kdenlive mailing list, and I found a bug in the
> kdenlive program when i was debugging a crash i was having on my
> computer. I'm not a member of the mlt devel list, and wasnt sure that
> the kdenlive list was the proper place to post this, so i figured just
> avoid the middle man.
> It basically has to do with the changes added to consumer_sdl.c in
> revision 1008. The changes that were added there (moving the
> initialization of the window_width and window_height variables from the
> consumer_sdl_init function to the consumer_start function) were not also
> added to the consumer_sdl_still.c file.
> Kdenlive was dying just after a project had loaded when it seg faulted
> in consumer_sdl_still.c on line 495 (the window_width and window_height
> were invalid).
> I've got a pretty good knowledge of the basics of MLT, but you might
> want to look over the patch to ensure that it fits in with how things work.
I was afraid I overlooked the impact of changes to mlt_consumer.c on some
consumers. Thank you for bringing it to my attention, and the patch appears
correct. Can you please post it to the relevant bug post in the kdenlive
forum for Jean-Michel to try? I know he is eager to resolve this.