Page MenuHomePhorge

Use immer for kazvcrypto sessions
ClosedPublic

Authored by tusooa on Tue, Sep 29, 7:54 PM.

Details

Summary

We used to use std containers in Crypto, which makes it expensive to copy. Now we store sessions using immer map and immer box, and we only need to copy those that changed and keep the rest. This should largely reduce the time needed in each sync cycle.

Type: fix

Test Plan

Check encryption/decryption in kazv still works.

Diff Detail

Repository
rL libkazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Sep 29, 8:12 PM
Harbormaster failed remote builds in B723: Diff 1118!

Both perf results and real-world testing show that it makes the client much more responsive

This revision is now accepted and ready to land.Sat, Oct 3, 7:51 AM
This revision was landed with ongoing or failed builds.Sat, Oct 3, 8:42 PM
This revision was automatically updated to reflect the committed changes.