DM: some refactoring

dm/DMReporter.cpp

Index: dm/DMReporter.cpp
diff --git a/dm/DMReporter.cpp b/dm/DMReporter.cpp
index 0e01d71f20c779397427671e5d6a5d83a36a5c9d..31310d1cd9a70cbf7610ed03ae7fa91c454182d8 100644
--- a/dm/DMReporter.cpp
+++ b/dm/DMReporter.cpp
@@ -12,7 +12,7 @@ void Reporter::updateStatusLine() const {
     }
 
     SkString status;
-    status.printf("\r\033[K%d / %d", this->finished(), this->started());
+    status.printf("\r\033[K%d tasks left", this->started() - this->finished());

[epoger, 2013/11/26 17:55:22]

This user prefers to see "completed X of Y", rather than just "(Y-X) remaining".

[bsalomon, 2013/11/26 18:21:52]

That is potentially misleading as Y grows in DM.

[mtklein, 2013/11/26 18:25:30]

Have you run this or just talking abstractly?  I'd be surprised if you really
preferred X of Y here.  Y increases as the program goes along, sometimes as fast
as X.  Y-X is a much better indicator of time remaining.

[epoger, 2013/11/26 18:34:48]

I have not run DM, and it's not a big deal to me either way... but given that Y
grows, it seems *especially* useful to know both X and Y at any given point.

Most users, given X and Y, will be able to calculate Y-X. :-)

At any rate... the more info, the better.

     const int failed = this->failed();
     if (failed > 0) {
         status.appendf(", %d failed", failed);