Page MenuHomePhabricator

Extend TemporaryMessageStore interface with private auxiliary methods to manipulate paths
ClosedPublic

Authored by marcin on Jul 7 2022, 5:38 AM.
Tags
None
Referenced Files
F3151813: D4472.id14698.diff
Tue, Nov 5, 2:52 AM
F3151806: D4472.id14469.diff
Tue, Nov 5, 2:48 AM
F3151799: D4472.id14638.diff
Tue, Nov 5, 2:43 AM
F3151765: D4472.id14259.diff
Tue, Nov 5, 2:36 AM
F3151747: D4472.id14427.diff
Tue, Nov 5, 2:32 AM
Unknown Object (File)
Tue, Oct 22, 5:25 PM
Unknown Object (File)
Mon, Oct 21, 8:20 PM
Unknown Object (File)
Oct 5 2024, 4:37 AM

Details

Summary

Adds and implements private methods to manipulate file names and create latest file to store messages

Test Plan

We can temporarily modify AppDelegate to invoke thos methods and check in the debugger that _getPath and getLockName return valid string and _updateCurrentStorage creates new file with expected name

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

marcin requested review of this revision.Jul 7 2022, 6:16 AM

Hey @marcinwasowicz, can you go through your entire diff stack and remove any reference to "Temporal", replacing it with "Temporary"?

Rename Temporal to Temporary

marcin retitled this revision from Extend TemporalMessageStore interface with private auxiliary methods to manipulate paths to Extend TemporaryMessageStore interface with private auxiliary methods to manipulate paths.

Hey @marcinwasowicz, can you go through your entire diff stack and remove any reference to "Temporal", replacing it with "Temporary"?

Temporal is renamed to Temporary in entire codebase of this stack as well as in differentials descriptions and titles.

tomek added inline comments.
native/ios/Comm/TemporaryMessageStorage/TemporaryMessageStorage.mm
42 ↗(On Diff #14276)

Maybe createNewStorage so that it is less confusing? The current name might indicate that we're renaming a storage.

This revision is now accepted and ready to land.Jul 8 2022, 3:08 AM
native/ios/Comm/TemporaryMessageStorage/TemporaryMessageStorage.mm
42 ↗(On Diff #14276)

Sure, I will do the refactor.

Rename new storage creation method and re-implement it to use fopen

This revision was landed with ongoing or failed builds.Jul 20 2022, 6:17 AM
This revision was automatically updated to reflect the committed changes.