Based on the patch from @Alberto Mardegan (mardy) (D2558), I have updated it to apply against current master, and set the option by default when adding a new scene strip so that the correct behaviour is used without the user needing to tick any boxes (but still maintaining compatibility with old files).
This comes handy!
Question is if it wouldn't be better to do timing conversion in do_versions and not add "Correct Timing" toggle to UI.
Would make UI and code a bit cleaner.
Is there any sensible reason to have such toggle?
I can't think of any reason to have the toggle - the option was added by Alberto, presumably at someone else's request. I think we should use the correct behaviour in all cases, since it was completely unusable before I can't think that anyone would have sub-scenes with animation in any position other than at frame 1.
Still, if it really mustn't change the behaviour of old files, having it done in versions and not in the UI is preferable. The checkbox is clutter and could be very confusing if accidentally un-ticked.
It didn't sound rude at all, no worries :-)
Preserving behaviour of existing files makes sense in a lot of cases, but not here in my opinion :)
I think we should always strive for that: you can never know whether some user already worked around the issue in some other means, and if we fix it unconditionally we'll most likely break his files. I think deciding to apply the fix based on the project's version is still the best idea; then of course it might not be worth the effort, but I think we should at least investigate the possibility.
Is this really required? Why would one disable option called "Correct"?
The comment should be explaining why something is done, not how. From quick reading i am still not sure why would order be important here.
And if it is, isn't it something to be moved to a dependency graph?