SolidWallOfCode commented on pull request #7485: URL: https://github.com/apache/trafficserver/pull/7485#issuecomment-786012103
Minor comments - `var.fetch_add(fetch_add(1, std::memory_order_relaxed)` can be replaced with `var++`. It's one of the nicer features of atomics that you can get pre or post increment values atomically via pre or post fix increment. `fetch_add` becomes useful when the value isn't exactly 1. ---------------------------------------------------------------- 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]
