Page MenuHomePhabricator

Wrap "<min, max>" chart domain pairs in an "Interval" class

Authored by epriestley on May 8 2019, 3:44 PM.



Ref T13279. Slightly simplify domain handling by putting all the "[x, y]" stuff in an Interval class. I'm planning to do something similar for ranges next, so this should make that easierr.

Test Plan

Viewed chart, saw same chart as before.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.May 8 2019, 3:44 PM
epriestley requested review of this revision.May 8 2019, 3:46 PM
amckinley accepted this revision.May 21 2019, 10:27 PM
This revision is now accepted and ready to land.May 21 2019, 10:27 PM