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
F15492683: D21733.id51793.diff
Sat, Apr 12, 5:07 PM
F15492200: D21733.id51791.diff
Sat, Apr 12, 12:15 PM
F15490903: D21733.id51792.diff
Fri, Apr 11, 9:36 PM
F15489004: D21733.id.diff
Fri, Apr 11, 5:08 AM
F15488424: D21733.diff
Thu, Apr 10, 10:12 PM
F15467467: D21733.id51793.diff
Thu, Apr 3, 3:21 PM
F15451668: D21733.id.diff
Fri, Mar 28, 11:16 PM
F15451667: D21733.id51793.diff
Fri, Mar 28, 11:16 PM
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.