clipboard: vector size only calculated once.

transition: check of t pointer inside the check sourcescommon: logical expression simplified.
2019-01-11 11:51:24 +01:00
parent ee019517b1
commit 652ac72c1c
3 changed files with 6 additions and 4 deletions
@@ -8,14 +8,16 @@ QVector<void*> clipboard;
 QVector<Transition*> clipboard_transitions;

 void clear_clipboard() {
-	for (int i=0;i<clipboard.size();i++) {
+	uint clipboard_size = clipboard.size();
+	for (int i=0;i<clipboard_size;i++) {
 		if (clipboard_type == CLIPBOARD_TYPE_CLIP) {
 			delete static_cast<Clip*>(clipboard.at(i));
 		} else if (clipboard_type == CLIPBOARD_TYPE_EFFECT) {
 			delete static_cast<Effect*>(clipboard.at(i));
 		}
 	}
-	for (int i=0;clipboard_transitions.size();i++) {
+	clipboard_size = clipboard_transitions.size();
+	for (int i=0;i<clipboard_size;i++) {
 		delete clipboard_transitions.at(i);
 	}
 	clipboard.clear();