You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2021/02/05 01:44:23 UTC

[GitHub] [trafficserver] bneradt commented on pull request #7486: slice/Range: change maxval to use defined value instead of numeric_limits

bneradt commented on pull request #7486:
URL: https://github.com/apache/trafficserver/pull/7486#issuecomment-773723000


   I believe per the standard numeric_limits are constexpr. https://en.cppreference.com/w/cpp/types/numeric_limits/max lists it as constexpr. I'd say numeric_limits are more idiomatic for C++ than the _MAX macros.
   
   Out of curiosity, @traeak , did you run into an issue with numeric_limits that motivated this?
   
   Thanks,


----------------------------------------------------------------
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