[PR] Cleanup of redundant HdrToken & WKS [trafficserver]

["zwoop (via GitHub)" <gi...@apache.org>]

zwoop opened a new pull request, #11751:
URL: https://github.com/apache/trafficserver/pull/11751

   (no comment)


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

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Cleanup of redundant HdrToken & WKS [trafficserver]

["zwoop (via GitHub)" <gi...@apache.org>]

zwoop merged PR #11751:
URL: https://github.com/apache/trafficserver/pull/11751


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

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Cleanup of redundant HdrToken & WKS [trafficserver]

["zwoop (via GitHub)" <gi...@apache.org>]

zwoop commented on PR #11751:
URL: https://github.com/apache/trafficserver/pull/11751#issuecomment-2321343145

   > This looks reasonable 👍
   > 
   > Probably, `_hdrtoken_strs` and `_hdrtoken_commonly_tokenized_strs` had different meanings in the past, but current code doesn't have it anymore.
   
   Yeh they did, _hdrtoken_commonly_tokenized_strs was smaller / different, but at some point (I can't find when / where) they were made identical. Maybe as part of us changing the hash map / searches ?


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

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org