Page MenuHomePhabricator

Use phutil_microseconds_since(...) to simplify some timing arithmetic
ClosedPublic

Authored by epriestley on Nov 8 2018, 3:33 PM.

Details

Summary

Depends on D19796. Simplify some timing code by using phutil_microseconds_since() instead of duplicate casting and arithmetic.

Test Plan

Grepped for 1000000 to find these. Pulled, pushed, made a conduit call. This isn't exhaustive but it should be hard for these to break in a bad way since they're all just diagnostic.

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.Nov 8 2018, 3:33 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Nov 8 2018, 3:34 PM
Harbormaster failed remote builds in B21119: Diff 47271!
epriestley requested review of this revision.Nov 8 2018, 4:54 PM
epriestley retitled this revision from Use phutil_microseconds_since(...) to simplify some to Use phutil_microseconds_since(...) to simplify some timing arithmetic.

Tests failed since we don't have phutil_microseconds_since() in master yet (until D19796).

amckinley accepted this revision.Nov 8 2018, 8:21 PM

Grepped for 1000000 to find these.

So we only found the callsites that do have the correct number of zeroes 😢

This revision is now accepted and ready to land.Nov 8 2018, 8:21 PM

hahaha

grep 10000 | grep -v 1000000 doesn't seem to turn up anything suspicious so I suspect we're okay.

This revision was automatically updated to reflect the committed changes.