Page MenuHomePhabricator

[native] Fork Olm

Authored by ashoat on Mar 16 2023, 7:34 PM.
Referenced Files
Unknown Object (File)
Sun, Sep 17, 7:17 PM
Unknown Object (File)
Sun, Aug 27, 7:20 PM
Unknown Object (File)
Sun, Aug 27, 7:20 PM
Unknown Object (File)
Sun, Aug 27, 7:20 PM
Unknown Object (File)
Sun, Aug 27, 4:41 AM
Unknown Object (File)
Sun, Aug 27, 2:39 AM
Unknown Object (File)
Sat, Aug 26, 5:17 PM
Unknown Object (File)
Sat, Aug 26, 3:35 PM



This is the native counterpart to D7089.

Depends on D7089

Test Plan
  1. Unit tests are added in D7090
  2. Also note that we successfully ran Olm C++ and JS test suites
  3. I also patched the web app to confirm it worked
  4. Confirmed the iOS app builds

Diff Detail

rCOMM Comm
Lint Not Applicable
Tests Not Applicable

Event Timeline

32 ↗(On Diff #23816)

This is not great... we're doing pointer arithmetic on a vector's data, and using an offset based on the number of chars we need to seek into the JSON blob

Note that this code isn't actually used in production yet and wasn't referenced outside of the file (deleted, more context on that in a separate comment)

991 ↗(On Diff #23816)

This might be controversial, but I deleted My understanding is that it's failing and I didn't really have context on why, but it's been failing for a while. It looks kind of messy and I'm not sure I have the cycles right now to fix it, and I don't want to block monthly goals on it. Open to creating a follow-up task if people want to keep this test around

This revision is now accepted and ready to land.Mar 17 2023, 12:49 PM
This revision was automatically updated to reflect the committed changes.