Alex Rousskov

Results 212 comments of Alex Rousskov

_Status update_: This is still on my to-do list. I hope to get back to this by the end of the week.

`S-waiting-for-PR`: The full effects described in this PR assume that #1222 fix has been merged. This PR changes work fine without that PR, but the cache.log file will remain lurking...

Thank you for keeping these v6-specific changes on the v6 branch!

> this PR is against v6; once accepted, it will need to be forward-ported to master ... but there is no rush with that second/porting step. We can even sync...

> Oh cool! I hadn’t noticed. Removing it everywhere That removal may help work around "getPod() used for a POD" build [failures](https://build.squid-cache.org/job/5-pr-auto/COMPILER=clang,OS=opensuse-leap,label=pr-merge-builder/1798/console) in master/v7 as well. Those failures are currently...

> Reviewer insists that we select a final and permanent Markdown syntax up-front and **only** use that one syntax forever, in all places Squid can or may now or in...

> I have intentionally avoided creating Async namespace, this PR touches so much code already. FWIW, I do not think creating Async namespace is a good idea (in any PR).

@sachin1gupta1980, thank you for opening this PR that addresses many #1398 concerns. Would you be able to help with the TODO items in the updated PR description? FWIW, for most...