Green-Sky/solanaceae_ngc_ft1
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

reserve memory to reduce number of allocations in hotspots

Browse Source 
especially on the sender side
This commit is contained in:
Green Sky 2024-03-16 11:30:55 +01:00
parent c03282eae8
commit 61b667a4aa
No known key found for this signature in database
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
solanaceae/ngc_ft1/ngcft1.cpp
View File

@ -98,6 +98,7 @@ bool NGCFT1::sendPKG_FT1_DATA(
// check header_size+data_size <= max pkg size // check header_size+data_size <= max pkg size
std::vector<uint8_t> pkg; std::vector<uint8_t> pkg;
pkg.reserve(2048); // saves a ton of allocations
pkg.push_back(static_cast<uint8_t>(NGCEXT_Event::FT1_DATA)); pkg.push_back(static_cast<uint8_t>(NGCEXT_Event::FT1_DATA));
pkg.push_back(transfer_id); pkg.push_back(transfer_id);
pkg.push_back(sequence_id & 0xff); pkg.push_back(sequence_id & 0xff);
@ -118,6 +119,7 @@ bool NGCFT1::sendPKG_FT1_DATA_ACK(
const uint16_t* seq_ids, size_t seq_ids_size const uint16_t* seq_ids, size_t seq_ids_size
) { ) {
std::vector<uint8_t> pkg; std::vector<uint8_t> pkg;
pkg.reserve(1+1+2*32); // 32acks in a single pkg should be unlikely
pkg.push_back(static_cast<uint8_t>(NGCEXT_Event::FT1_DATA_ACK)); pkg.push_back(static_cast<uint8_t>(NGCEXT_Event::FT1_DATA_ACK));
pkg.push_back(transfer_id); pkg.push_back(transfer_id);
@ -578,7 +580,7 @@ bool NGCFT1::onEvent(const Events::NGCEXT_ft1_init_ack& e) {
bool NGCFT1::onEvent(const Events::NGCEXT_ft1_data& e) { bool NGCFT1::onEvent(const Events::NGCEXT_ft1_data& e) {
#if !NDEBUG #if !NDEBUG
std::cout << "NGCFT1: FT1_DATA\n"; //std::cout << "NGCFT1: FT1_DATA\n";
#endif #endif
if (e.data.empty()) { if (e.data.empty()) {