maskit commented on a change in pull request #7079:
URL: https://github.com/apache/trafficserver/pull/7079#discussion_r464767114
##########
File path: iocore/net/quic/QUICTransportParameters.h
##########
@@ -61,7 +61,7 @@ class QUICTransportParameterId
}
bool
- operator==(uint16_t &x) const
+ operator==(const uint16_t &x) const
Review comment:
I guess you can find a lot of this kind of stuff in QUIC module. It'd be
great if you could make PRs to correct use of references. This change is fine
with me because it's a right thing to do in terms of const-ify.
----------------------------------------------------------------
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]
