X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=pv%2Fdata%2Flogicsegment.cpp;h=739b2b9ee2a97652de2d4d4e3530041c1613af2c;hb=HEAD;hp=354c14f85d8bef035fd3a184757aabfb5d304c0b;hpb=04b0467582d8dfa989bb38fe18bfb1be20cea456;p=pulseview.git

diff --git a/pv/data/logicsegment.cpp b/pv/data/logicsegment.cpp
index 354c14f8..739b2b9e 100644
--- a/pv/data/logicsegment.cpp
+++ b/pv/data/logicsegment.cpp
@@ -363,26 +363,28 @@ void LogicSegment::append_payload(void *data, uint64_t data_size)
 			prev_sample_count + 1, prev_sample_count + 1);
 }
 
-void LogicSegment::append_subsignal_payload(unsigned int index, void *data, uint64_t data_size)
+void LogicSegment::append_subsignal_payload(unsigned int index, void *data,
+	uint64_t data_size, vector<uint8_t>& destination)
 {
-	static vector<uint8_t> merged_data;  // Using static also places it on the heap
-
-	for (uint64_t i = 0; i < data_size * unit_size_; i++)
-		merged_data.emplace_back(0);
+	if (index == 0)
+		destination.resize(data_size * unit_size_, 0);
 
 	// Set the bits for this sub-signal where needed
 	// Note: the bytes in *data must either be 0 or 1, nothing else
-	unsigned int index_byte = index / 8;
+	unsigned int index_byte_offs = index / 8;
+	uint8_t* output_data = destination.data() + index_byte_offs;
+	uint8_t* input_data = (uint8_t*)data;
+
 	for (uint64_t i = 0; i < data_size; i++) {
-		unsigned int offs = i * unit_size_ + index_byte;
-		uint8_t* data_byte = merged_data.data() + offs;
-		*data_byte |= *((uint8_t*)data + i) << index;
+		assert((i * unit_size_ + index_byte_offs) < destination.size());
+		*output_data |= (input_data[i] << index);
+		output_data += unit_size_;
 	}
 
 	if (index == owner_.num_channels() - 1) {
 		// We gathered sample data of all sub-signals, let's append it
-		append_payload(merged_data.data(), merged_data.size());
-		merged_data.clear();
+		append_payload(destination.data(), destination.size());
+		destination.clear();
 	}
 }