Page MenuHomePhabricator

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

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

Details

Summary

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

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

Event Timeline

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