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

   #10711 removed the necessity of linking libhttp, but I found header files 
are still included and inline functions make some magic.
   
   To not include the header files, this PR moves `calculate_document_age` from 
`HttpTransactHeaders` to `HttpTransactCache` which has other calculation 
functions.
   
   
   There is actually one more place that includes a header file in proxy/http
   ```
   $ git grep "#include" src/iocore/cache | grep "proxy/http"
   src/iocore/cache/CacheRead.cc:#include "proxy/http/HttpCacheSM.h" //Added to 
get the scope of HttpCacheSM object.
   ```
   And it's used here
   
https://github.com/apache/trafficserver/blob/48333c8bc13102b4d3b570bfb25e8d1109f373e9/src/iocore/cache/CacheRead.cc#L171-L172
   I'll try to remove this on another PR to make this one simple.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to