fluyu commented on issue #322:
URL: 
https://github.com/apache/rocketmq-client-cpp/issues/322#issuecomment-663507546


   > I think u are right but use getCacheMsgCount() to check cache is empty is 
better, and getCacheMsgCount need to change to below:
   > 
   > ```
   > int PullRequest::getCacheMsgCount() {
   >   boost::lock_guard<boost::mutex> lock(m_pullRequestLock);
   >   return m_msgTreeMap.size() + m_msgTreeMapTemp.size();
   > }
   > ```
   
   Thanks for the suggestion, followed and verified, works like a charm.
   
   
   > @fluyu Could you pull a request to fix this issue?
   
   My pleasure and done. #324 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to