Add function get_null_fd(), to reduce code redundancy.

[openvswitch] / lib / coverage.c

diff --git a/lib/coverage.c b/lib/coverage.c
index e86aa0299464e4684afda1fbd50fa6bb224b366d..ba5d68d84751512e69063e4fc31361f388e9fe48 100644 (file)
--- a/lib/coverage.c
+++ b/lib/coverage.c
@@ -36,18 +36,10 @@ coverage_unixctl_log(struct unixctl_conn *conn, const char *args UNUSED)
     unixctl_command_reply(conn, 200, NULL);
 }
 
-static void
-coverage_unixctl_clear(struct unixctl_conn *conn, const char *args UNUSED)
-{
-    coverage_clear();
-    unixctl_command_reply(conn, 200, NULL);
-}
-
 void
 coverage_init(void)
 {
     unixctl_command_register("coverage/log", coverage_unixctl_log);
-    unixctl_command_register("coverage/clear", coverage_unixctl_clear);
 }
 
 /* Sorts coverage counters in descending order by count, within equal counts
@@ -142,8 +134,8 @@ coverage_log(enum vlog_level level, bool suppress_dups)
         return;
     }
 
+    hash = coverage_hash();
     if (suppress_dups) {
-        hash = coverage_hash();
         if (coverage_hit(hash)) {
             VLOG(level, "Skipping details of duplicate event coverage for "
                  "hash=%08"PRIx32" in epoch %u", hash, epoch);