Paths

Table of Contentst

Differential D19796

Add a convenience function for computing elapsed microtime deltas
ClosedPublic
Actions

Authored by epriestley on Nov 8 2018, 2:13 PM.

Tags

None

Referenced Files

	F13529507: D19796.diff
	Fri, Jul 26, 10:27 PM

	Unknown Object (File)
	Sun, Jul 21, 4:36 AM

	Unknown Object (File)
	Wed, Jul 17, 3:09 AM

	Unknown Object (File)
	Fri, Jul 12, 2:38 PM

	Unknown Object (File)
	Fri, Jul 12, 6:12 AM

	Unknown Object (File)
	Tue, Jul 9, 2:00 AM

	Unknown Object (File)
	Sat, Jul 6, 10:43 AM

	Unknown Object (File)
	Fri, Jul 5, 2:07 PM

View All 98 Files

Subscribers

None

Details

Reviewers

Commits

rPHUb29d76e1709e: (stable) Promote 2018 Week 45
rPHU335a9f8c2dce: Add a convenience function for computing elapsed microtime deltas

Summary

See D19780. We have a reasonable amount of 1000000 * ($t_end - $t_start) stuff in the codebase that we could simplify. In particular, it would be easy to have too many or too few 0s somewhere and escape notice.

I landed on this method signature instead of phutil_microseconds_between(x, y) since I think it would be hard to remember if parameter order is start, end (chronological) or end, start (like arithmetic subtraction). It should be a little simpler to use in most cases, too.

Test Plan

$ cat test.php
<?php

require_once 'scripts/init/init-script.php';

$start = microtime(true);

sleep(1);

echo phutil_microseconds_since($start);
echo "\n";

$ php -f test.php
1001921

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Nov 8 2018, 2:13 PM

Harbormaster completed remote builds in B21118: Diff 47270.Nov 8 2018, 2:14 PM

epriestley requested review of this revision.Nov 8 2018, 2:14 PM

epriestley added a child revision: D19797: Use phutil_microseconds_since(...) to simplify some timing arithmetic.Nov 8 2018, 3:33 PM

epriestley mentioned this in D19797: Use phutil_microseconds_since(...) to simplify some timing arithmetic.Nov 8 2018, 4:55 PM

amckinley accepted this revision.Nov 8 2018, 7:53 PM

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

Closed by commit rPHU335a9f8c2dce: Add a convenience function for computing elapsed microtime deltas (authored by epriestley). · Explain WhyNov 8 2018, 8:51 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

utils/

23 lines

Diff 47274

src/__phutil_library_map__.php

Loading...

src/utils/utils.php

Loading...