Print runtimes for each individual polling and discovery module. #2713

Merged
merged 2 commits into from Jan 8, 2016

Projects

None yet

3 participants

@SaaldjorMike
Member

No description provided.

@SaaldjorMike SaaldjorMike Print runtimes for each individual polling and discovery module.
51d2e32
@f0o
Member
f0o commented Jan 8, 2016

Substitute utime() with microtime(true) please

@SaaldjorMike
Member

@f0o Will do. Should we look at removing the utime() function entirely then (in separate PR ofc), as it is used elsewhere in the same functions i added the lines?

@SaaldjorMike SaaldjorMike Replace utime() for microtime(true)
a1261a0
@f0o
Member
f0o commented Jan 8, 2016

Afaik utime isnt even native php. I got no idea what Adam et. al. did in it when getting precision time is as simple as using microtime(true)

@f0o
Member
f0o commented Jan 8, 2016
function utime() {
    $time = explode(' ', microtime());
    $usec = (double) $time[0];
    $sec  = (double) $time[1];
    return ($sec + $usec);

}

That's a lot of code for similar result like microtime(true)

@f0o f0o merged commit 19471a3 into librenms:master Jan 8, 2016

2 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer 4 new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment