From 54ee1372c4254e185e83c8eb9329ace9704664c6 Mon Sep 17 00:00:00 2001
From: Dan Engelbrecht <dan.engelbrecht@epicgames.com>
Date: Mon, 30 Sep 2024 09:27:36 +0200
Subject: optimize startup time (#175)

* use tsl::robin_set for BlockIndexSet
don't calculate full block location when only block index is needed
* don't copy visitor function
* reserve space for attachments
---
 src/zencore/compactbinary.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/zencore/compactbinary.cpp')

diff --git a/src/zencore/compactbinary.cpp b/src/zencore/compactbinary.cpp
index 029889e85..adccaba70 100644
--- a/src/zencore/compactbinary.cpp
+++ b/src/zencore/compactbinary.cpp
@@ -407,7 +407,7 @@ CbFieldView::CbFieldView(const void* DataPointer, CbFieldType FieldType)
 }
 
 void
-CbFieldView::IterateAttachments(std::function<void(CbFieldView)> Visitor) const
+CbFieldView::IterateAttachments(const std::function<void(CbFieldView)>& Visitor) const
 {
 	switch (CbFieldTypeOps::GetType(Type))
 	{
@@ -1170,7 +1170,7 @@ template class TCbFieldIterator<CbField>;
 
 template<typename FieldType>
 void
-TCbFieldIterator<FieldType>::IterateRangeAttachments(std::function<void(CbFieldView)> Visitor) const
+TCbFieldIterator<FieldType>::IterateRangeAttachments(const std::function<void(CbFieldView)>& Visitor) const
 {
 	if (CbFieldTypeOps::HasFieldType(FieldType::GetType()))
 	{
-- 
cgit v1.2.3