[Bf-blender-cvs] [d33e7152597] master: Fix T77669: Copy-pasting strip twice crashes Blender

Richard Antalik noreply at git.blender.org
Fri Jun 26 15:51:21 CEST 2020


Commit: d33e7152597361490eb8240ce931df0ec7f12ea5
Author: Richard Antalik
Date:   Fri Jun 26 15:49:11 2020 +0200
Branches: master
https://developer.blender.org/rBd33e7152597361490eb8240ce931df0ec7f12ea5

Fix T77669: Copy-pasting strip twice crashes Blender

In rBeaee2b411935 I removed BKE_sequencer_base_clipboard_pointers_store from
paste function, because I wanted pasted strip to always reference copied
strip, not recently pasted one. This worked well with ordinary strips, but not
if they reference ID.

Add back BKE_sequencer_base_clipboard_pointers_store call.

I thought that always referencing original strip would be more correct approach,
but it doesn't matter now. Referencing recently pasted seems to be cleanest way
to do this.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D8129

===================================================================

M	source/blender/editors/space_sequencer/sequencer_edit.c

===================================================================

diff --git a/source/blender/editors/space_sequencer/sequencer_edit.c b/source/blender/editors/space_sequencer/sequencer_edit.c
index 60d98407025..2c8c928c65c 100644
--- a/source/blender/editors/space_sequencer/sequencer_edit.c
+++ b/source/blender/editors/space_sequencer/sequencer_edit.c
@@ -3453,6 +3453,7 @@ static int sequencer_paste_exec(bContext *C, wmOperator *UNUSED(op))
    * on the actual data-blocks. */
   BKE_sequencer_base_clipboard_pointers_restore(&seqbase_clipboard, bmain);
   BKE_sequence_base_dupli_recursive(scene, scene, &nseqbase, &seqbase_clipboard, 0, 0);
+  BKE_sequencer_base_clipboard_pointers_store(bmain, &seqbase_clipboard);
 
   iseq_first = nseqbase.first;



More information about the Bf-blender-cvs mailing list