Page MenuHomePhabricator

Provide an API for parsing swap information from "/proc/meminfo"
ClosedPublic

Authored by epriestley on Nov 22 2021, 1:30 PM.
Tags
None
Referenced Files
F15451668: D21733.id.diff
Fri, Mar 28, 11:16 PM
F15451667: D21733.id51793.diff
Fri, Mar 28, 11:16 PM
F15451666: D21733.id51791.diff
Fri, Mar 28, 11:16 PM
F15451665: D21733.id51792.diff
Fri, Mar 28, 11:16 PM
F15451647: D21733.diff
Fri, Mar 28, 11:08 PM
F15445824: D21733.id51793.diff
Thu, Mar 27, 4:10 PM
F15443995: D21733.id51793.diff
Thu, Mar 27, 7:14 AM
F15422695: D21733.id51793.diff
Sat, Mar 22, 10:14 AM
Subscribers
None

Diff Detail

Repository
rARC Arcanist
Branch
mem1
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsrc/filesystem/memory/__tests__/ArcanistHostMemorySnapshotTestCase.php:28XHP132Partial Catch
Unit
Tests Passed
Build Status
Buildable 25585
Build 35391: Run Core Tests
Build 35390: arc lint + arc unit

Event Timeline

Satisfy lint by catching and re-throwing Throwable.

It would be slightly more appropriate for this class to throw a more tailored exception, but I don't think there's a suitable existing exception and this already feels very heavy for its actual use case.

This overlaps somewhat with PhutilSystem::getSystemMemoryInformation(), but that mechanism is narrower and also doesn't really work in practice, so it would probably be reasonable to remove it entirely.