Yat Ho

Results 355 comments of Yat Ho

> Just registering (and acknowleding again that my C++fu is weak) that moving ALL writes to the session work queue feels like a pretty dramatic change. Deferring to your skills...

@reardonia 0. There's actually 2 parts to this PR. (1) Rework active request tracking to save CPU (2) Flush the outgoing buffer as soon as the session thread is available....

> It seems that, on first principles, using the event queue will increase latency vs not using that queue. Yes, you are right. Sorry for misunderstanding earlier. > I think...

@gruve-p Unfortunately no. It's based on an earlier major refactor that is very far ahead of the `4.0.x` branch @ckerr Ready for review/merge.

> The plan is to have Inspector pane take over all screen on small screens. Well, in that case you can just set `position: absolute` and `top: 0`, no?

@Rukario Please include me as the commit author for my changes, thanks.

Looks like `package.json.buildonly` got updated by the CI just fine. 👍🏻

@ckerr The WebUI build in the `4.1.0-beta.2` tag does not match the source code because 7b83c7d6250423ee0b7cdaf3ad95fa25be0e1190 (contains WebUI changes) is commited after 40d73978f7e92d9dbffb6841733d0902279101d7. This PR seems ready to merge now...

@nengxu Thanks for reporting. I'm currently focused on issue 6986, I will get around to this when I have the time, or hopefully somebody else can do it before I...

Just tested on 3e5a77d17603dd6548add3c5b233a4e87d38c577 and cannot reproduce. ![test](https://github.com/user-attachments/assets/3b93ad6f-e421-47ec-99e3-9f03dcd67d22) The binary was built by myself, and every setting was on default. ``` cmake -G Ninja -B build_reldeb -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_TESTS=NO -DRUN_CLANG_TIDY=NO -DCMAKE_TOOLCHAIN_FILE=C:\vcpkg\scripts\buildsystems\vcpkg.cmake...