This is an automated email from the ASF dual-hosted git repository.
xyz pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/pulsar-client-cpp.git
The following commit(s) were added to refs/heads/main by this push:
new 5451797 fix: remove unnecessary copies (#499)
5451797 is described below
commit 54517973f8107c6d17b993e7b9da77c1162d8e26
Author: Guangyang Deng <[email protected]>
AuthorDate: Tue Aug 26 10:06:13 2025 +0800
fix: remove unnecessary copies (#499)
---
lib/NegativeAcksTracker.cc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/NegativeAcksTracker.cc b/lib/NegativeAcksTracker.cc
index 9dc64cd..ba008a8 100644
--- a/lib/NegativeAcksTracker.cc
+++ b/lib/NegativeAcksTracker.cc
@@ -88,9 +88,9 @@ void NegativeAcksTracker::handleTimer(const ASIO_ERROR &ec) {
break;
}
- auto ledgerMap = it->second;
+ const auto &ledgerMap = it->second;
for (auto ledgerIt = ledgerMap.begin(); ledgerIt != ledgerMap.end();
++ledgerIt) {
- auto entrySet = ledgerIt->second;
+ const auto &entrySet = ledgerIt->second;
for (auto setIt = entrySet.begin(); setIt != entrySet.end();
++setIt) {
messagesToRedeliver.insert(
MessageIdBuilder().ledgerId(ledgerIt->first).entryId(*setIt).build());