Page MenuHome

VSE: Delete Strip and Data in one step
ClosedPublic

Authored by Antonio Vazquez (antoniov) on Apr 28 2022, 4:58 PM.

Details

Summary

Actually, delete the strip only deletes the container, but not the linked data. This patch adds the option to delete the data also. This is very handy for storyboarding.

Note: Actual patch only supports Scene strips.

Diff Detail

Repository
rB Blender

Event Timeline

Antonio Vazquez (antoniov) requested review of this revision.Apr 28 2022, 4:58 PM
Antonio Vazquez (antoniov) created this revision.
  • Fix compiler warning
  • Remove not supported types
Antonio Vazquez (antoniov) retitled this revision from Sequencer: Delete Strip and Data in one step to VSE: Delete Strip and Data in one step.May 5 2022, 5:39 PM
  • Replace and Data with & Data
Richard Antalik (ISS) added inline comments.
source/blender/editors/space_sequencer/sequencer_edit.c
1723–1737

This function looks quite weird - it returns value, but this is not used anywhere. Is that for any future work? Also there is switch statement with one branch only. Early return on non-scene strip would save 3 indentation levels.

This revision is now accepted and ready to land.May 16 2022, 3:07 PM