From f7a6d07d715ac401c3d04d85bda878da9a67be6b Mon Sep 17 00:00:00 2001
From: Paul Beckingham <paul@beckingham.net>
Date: Tue, 10 Sep 2013 23:16:03 -0400
Subject: [PATCH] Code Cleanup

- The 'diagnostics' command no longer generates 1000 UUIDs as a uniqueness
  test, because libuuid is now mandatory.
---
 NEWS                            |  2 ++
 src/commands/CmdDiagnostics.cpp | 18 ------------------
 2 files changed, 2 insertions(+), 18 deletions(-)

diff --git a/NEWS b/NEWS
index 0d1a0fb93..12551ca1d 100644
--- a/NEWS
+++ b/NEWS
@@ -32,6 +32,8 @@ Newly deprecated features in taskwarrior 2.3.0
   - The 'push', 'pull' and 'merge' commands will be removed in a subsequent
     release.  The new 'sync' command, in conjunction with the Task Server is the
     new sync mechanism.
+  - The 'diagnostics' command no longer generates 1000 UUIDs as a uniqueness
+    test, because libuuid is now mandatory.
 
 ---
 
diff --git a/src/commands/CmdDiagnostics.cpp b/src/commands/CmdDiagnostics.cpp
index a76cb81ac..e91f80b10 100644
--- a/src/commands/CmdDiagnostics.cpp
+++ b/src/commands/CmdDiagnostics.cpp
@@ -300,24 +300,6 @@ int CmdDiagnostics::execute (std::string& output)
   out << bold.colorize (STRING_CMD_DIAG_TESTS)
       << "\n";
   {
-    out << "   UUID gen: ";
-    std::vector <std::string> uuids;
-    std::string id;
-    for (int i = 0; i < 1000; i++)
-    {
-      id = uuid ();
-      if (std::find (uuids.begin (), uuids.end (), id) != uuids.end ())
-      {
-        out << format (STRING_CMD_DIAG_UUID_BAD, i) << "\n";
-        break;
-      }
-      else
-        uuids.push_back (id);
-    }
-
-    if (uuids.size () >= 1000)
-      out << STRING_CMD_DIAG_UUID_GOOD << "\n";
-
     // Determine terminal details.
     const char* term = getenv ("TERM");
     out << "      $TERM: "