Page MenuHomePhabricator

[native] Memoize `ConnectedRemoteImage`
ClosedPublic

Authored by atul on Sep 14 2023, 7:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 2, 8:25 PM
Unknown Object (File)
Thu, Jun 27, 9:41 AM
Unknown Object (File)
Wed, Jun 26, 4:03 PM
Unknown Object (File)
Wed, Jun 26, 12:56 PM
Unknown Object (File)
Wed, Jun 26, 9:45 AM
Unknown Object (File)
Wed, Jun 19, 9:15 PM
Unknown Object (File)
Tue, Jun 18, 7:18 AM
Unknown Object (File)
Sat, Jun 15, 8:18 AM
Subscribers

Details

Summary

Similar to D9213, the profiler isn't providing much information here so I'm memoizing this on a whim to see if anything changes. Doubt this changes anything at all.


Depends on D9213

Test Plan

ConnectedRemoteImage continues to look as expected.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul published this revision for review.Sep 14 2023, 7:44 PM

If the performance isn't impacted much one way or the other, is the memory usage of memoizing this not noticeable as well?

This revision is now accepted and ready to land.Sep 15 2023, 2:21 PM
In D9214#271139, @rohan wrote:

If the performance isn't impacted much one way or the other, is the memory usage of memoizing this not noticeable as well?

Memory usage of these memoizations is pretty much trivial

This revision was landed with ongoing or failed builds.Sep 17 2023, 3:31 PM
This revision was automatically updated to reflect the committed changes.