Alex Rousskov

> the context info related to parsing is intended to be held by the Parser object. In later PR a Parser instance may be passed to any of the low-level...

> 4a837ae: Update TcpLogger to new event queue API FWIW, I do not recommend propagating changes to other use cases until we are done with the API and first use...

@amk1969, thank you for testing this and suggesting fixes! Clearly, a lot more needs to be done before this code becomes production-worthy. I do not plan to resume working on...

I am posting this unpolished Draft PR to ask the following two related questions: 1. To fix foreground indexing, we have to add a few IPC messages. We have discovered...

Thank you for a prompt and clear answer. We will probably split in two or three parts. Until then, please ignore this draft PR.

**Status update**: > > split in two or three parts > > I separated the first part as [PR756](https://github.com/squid-cache/squid/pull/756). PR #756 has been merged. The second prerequisite pull request --...

@kinkie, another strange Jenkings [failure](https://build.squid-cache.org/job/5-pr-test/COMPILER=clang,OS=debian-unstable,label=docker-build-host/4612/console), but this one looks different. I failed to quickly find anything clearly relevant for "Error 141" but I am not even sure which program produces...

> I do not see why Optional is necessary here. We should use Optional here because `Optional` is the overall best way to represent absence of a stack-allocated `X`. >...

> > We should use Optional here because `Optional` is the overall best way to represent absence of a stack-allocated `X`. > Except when x has its own built-in indicator...

Clearing for merge but redoing all tests due to a Jenkins "illegal instruction" [failure](https://build.squid-cache.org/job/5-pr-auto/COMPILER=gcc,OS=ubuntu-jammy,label=docker-build-host/1246/console) during the last round of staged commit tests.