[GitHub] [trafficserver] sudheerv commented on pull request #7934: Internal DLL container forward iterators

[GitBox <gi...@apache.org>]

sudheerv commented on pull request #7934:
URL: https://github.com/apache/trafficserver/pull/7934#issuecomment-858827508


   > I wouldn't do that. I'd do something (in `Trie`) like
   > 
   > ```
   > using cont_iterator = Queue<T>::const_iterator;
   > const_iterator begin() const { m_value_list.begin(); }
   > const_iterator end() const { m_value_list.end(); }
   > ```
   > 
   > One of the points of having an iterator class is to keep the container protected.
   
   Makes sense. This will work too.


-- 
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:
users@infra.apache.org