Patches to fix race conditions in auto-alignment feature

[farid abdelnour]

Hi Raphaël,

Welcome to the community!

Is your patch a response to the recent post we published [1] or did you get
here by other means?

Over at phabricator you'll also be able to get a glimpse of the refactoring
work being made, exiting times coming to Kdenlive.

[1] https://kdenlive.org/2017/03/kdenlive-status-update/

Thanks for the contribution :)

Cheers

2017-04-14 6:55 GMT-03:00 alcinos <french.ebook.lover at gmail.com>:

> Hi Raphaël,
>
> Thank you for your contribution! Your PR looks good to me so far.
>
> Please note however that KDE (and thus Kdenlive) uses Phabricator for the
> review and merge process (github is only a mirror). Would it be possible
> for you to submit it there? (How to: https://community.kde.org/
> Infrastructure/Phabricator#Posting_a_Patch, link to the phabricator:
> https://phabricator.kde.org/)
> You can set mardelle and alcinos as reviewers.
>
> Cheers,
>
> alcinos
>
>
>
> 2017-04-13 23:23 GMT+02:00 Raphaël Marinier <raphael.marinier at gmail.com>:
>
>> Hi all,
>>
>> I am a new user of kdenlive, and noticed arbitrary alignment of tracks
>> from time to time when using the auto alignment feature.
>> I investigated and found one race condition (and a couple of smaller
>> issues). I propose a fix with pull request
>> https://github.com/KDE/kdenlive/pull/6.
>>
>> Please have a look and tell me what you think,
>>
>> Thanks,
>>
>> Raphael M.
>>
>
>


-- 
1111.1010.r.i.1101|n.o.i.s.1110|i.m.1010.g.1110|مقاومة
fsf member #5439
usuario GNU/Linux #471966
|_|0|_|
|_|_|0|
|0|0|0|
<a href="http://www.gunga.com.br">gunga</a>
<a href="http://www.tempoecoarte.com.br">tempoecoarte</a>
<a href="http://www.atelier-labs.org">atelier-labs</a>
<a href="http://www.mocambos.net">rede mocambos</a>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdenlive/attachments/20170414/3b2b0cbc/attachment.html>