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());

Reply via email to