Start using Timing class in more places

... which allows for removing some duplicate code and streamlining consistent display of timing information.

Author: jrfnl

.github/workflows/build-phar.yml

@@ -16,6 +16,7 @@ on:
       - 'src/Exceptions/RuntimeException.php'
       - 'src/Exceptions/TokenizerException.php'
       - 'src/Tokenizers/PHP.php'
+      - 'src/Util/Timing.php'
       - 'src/Util/Tokens.php'
   pull_request:
     paths:
@@ -27,6 +28,7 @@ on:
       - 'src/Exceptions/RuntimeException.php'
       - 'src/Exceptions/TokenizerException.php'
       - 'src/Tokenizers/PHP.php'
+      - 'src/Util/Timing.php'
       - 'src/Util/Tokens.php'
 
   # Allow manually triggering the workflow.

scripts/build-phar.php

@@ -17,6 +17,7 @@
 use PHP_CodeSniffer\Exceptions\RuntimeException;
 use PHP_CodeSniffer\Exceptions\TokenizerException;
 use PHP_CodeSniffer\Tokenizers\PHP;
+use PHP_CodeSniffer\Util\Timing;
 use PHP_CodeSniffer\Util\Tokens;
 
 error_reporting(E_ALL);
@@ -78,7 +79,7 @@ function stripWhitespaceAndComments($fullpath, $config)
 }//end stripWhitespaceAndComments()
 
 
-$startTime = microtime(true);
+Timing::startTiming();
 
 $scripts = [
     'phpcs',
@@ -170,14 +171,7 @@ function stripWhitespaceAndComments($fullpath, $config)
     echo 'done'.PHP_EOL;
 }//end foreach
 
-$timeTaken = ((microtime(true) - $startTime) * 1000);
-if ($timeTaken < 1000) {
-    $timeTaken = round($timeTaken);
-    echo "DONE in {$timeTaken}ms".PHP_EOL;
-} else {
-    $timeTaken = round(($timeTaken / 1000), 2);
-    echo "DONE in $timeTaken secs".PHP_EOL;
-}
+Timing::printRunTime();
 
 echo PHP_EOL;
 echo 'Filesize generated phpcs.phar file: '.filesize(dirname(__DIR__).'/phpcs.phar').' bytes'.PHP_EOL;

src/Reports/Cbf.php

@@ -15,6 +15,7 @@
 
 use PHP_CodeSniffer\Exceptions\DeepExitException;
 use PHP_CodeSniffer\Files\File;
+use PHP_CodeSniffer\Util\Timing;
 use PHP_CodeSniffer\Util\Writers\StatusWriter;
 
 class Cbf implements Report
@@ -73,14 +74,7 @@ public function generateFileReport($report, File $phpcsFile, $showSources=false,
                 StatusWriter::forceWrite('DONE', 0, 0);
             }
 
-            $timeTaken = ((microtime(true) - $startTime) * 1000);
-            if ($timeTaken < 1000) {
-                $timeTaken = round($timeTaken);
-                StatusWriter::forceWrite(" in {$timeTaken}ms");
-            } else {
-                $timeTaken = round(($timeTaken / 1000), 2);
-                StatusWriter::forceWrite(" in $timeTaken secs");
-            }
+            StatusWriter::forceWrite(' in '.Timing::getHumanReadableDuration(Timing::getDurationSince($startTime)));
         }
 
         if ($fixed === true) {

src/Reports/Code.php

@@ -12,6 +12,7 @@
 use Exception;
 use PHP_CodeSniffer\Files\File;
 use PHP_CodeSniffer\Util\Common;
+use PHP_CodeSniffer\Util\Timing;
 use PHP_CodeSniffer\Util\Writers\StatusWriter;
 
 class Code implements Report
@@ -65,14 +66,7 @@ public function generateFileReport($report, File $phpcsFile, $showSources=false,
             }
 
             if (PHP_CODESNIFFER_VERBOSITY === 1) {
-                $timeTaken = ((microtime(true) - $startTime) * 1000);
-                if ($timeTaken < 1000) {
-                    $timeTaken = round($timeTaken);
-                    StatusWriter::forceWrite("DONE in {$timeTaken}ms");
-                } else {
-                    $timeTaken = round(($timeTaken / 1000), 2);
-                    StatusWriter::forceWrite("DONE in $timeTaken secs");
-                }
+                StatusWriter::forceWrite('DONE in '.Timing::getHumanReadableDuration(Timing::getDurationSince($startTime)));
             }
 
             $tokens = $phpcsFile->getTokens();

src/Reports/Diff.php

@@ -10,6 +10,7 @@
 namespace PHP_CodeSniffer\Reports;
 
 use PHP_CodeSniffer\Files\File;
+use PHP_CodeSniffer\Util\Timing;
 use PHP_CodeSniffer\Util\Writers\StatusWriter;
 
 class Diff implements Report
@@ -51,14 +52,7 @@ public function generateFileReport($report, File $phpcsFile, $showSources=false,
             $phpcsFile->parse();
 
             if (PHP_CODESNIFFER_VERBOSITY === 1) {
-                $timeTaken = ((microtime(true) - $startTime) * 1000);
-                if ($timeTaken < 1000) {
-                    $timeTaken = round($timeTaken);
-                    StatusWriter::write("DONE in {$timeTaken}ms");
-                } else {
-                    $timeTaken = round(($timeTaken / 1000), 2);
-                    StatusWriter::write("DONE in $timeTaken secs");
-                }
+                StatusWriter::write('DONE in '.Timing::getHumanReadableDuration(Timing::getDurationSince($startTime)));
             }
 
             $phpcsFile->fixer->startFile($phpcsFile);

src/Runner.php

@@ -612,14 +612,7 @@ public function processFile($file)
             $file->process();
 
             if (PHP_CODESNIFFER_VERBOSITY > 0) {
-                $timeTaken = ((microtime(true) - $startTime) * 1000);
-                if ($timeTaken < 1000) {
-                    $timeTaken = round($timeTaken);
-                    StatusWriter::write("DONE in {$timeTaken}ms", 0, 0);
-                } else {
-                    $timeTaken = round(($timeTaken / 1000), 2);
-                    StatusWriter::write("DONE in $timeTaken secs", 0, 0);
-                }
+                StatusWriter::write('DONE in '.Timing::getHumanReadableDuration(Timing::getDurationSince($startTime)), 0, 0);
 
                 if (PHP_CODESNIFFER_CBF === true) {
                     $errors = $file->getFixableCount();