You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2017/05/26 23:29:59 UTC

[6/6] mesos git commit: Removed a use of the 'default' switch case.

Removed a use of the 'default' switch case.

For enums, we instead rely on the compiler to warn us when we
miss a switch case. With the presence of the 'default' case,
this doesn't occur.

Review: https://reviews.apache.org/r/59591


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/b6b23443
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/b6b23443
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/b6b23443

Branch: refs/heads/master
Commit: b6b23443a393bb31ab7730f43f6bcd017ffc8c51
Parents: 3f029d0
Author: Benjamin Mahler <bm...@apache.org>
Authored: Thu May 25 15:25:42 2017 -0700
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Fri May 26 16:29:44 2017 -0700

----------------------------------------------------------------------
 src/slave/slave.cpp | 4 ----
 1 file changed, 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/b6b23443/src/slave/slave.cpp
----------------------------------------------------------------------
diff --git a/src/slave/slave.cpp b/src/slave/slave.cpp
index 2835fc2..2eb9034 100644
--- a/src/slave/slave.cpp
+++ b/src/slave/slave.cpp
@@ -4149,10 +4149,6 @@ void Slave::reregisterExecutor(
       // should be shutdown if it hasn't received any tasks.
       break;
     }
-    default:
-      LOG(FATAL) << "Executor " << *executor << " is in unexpected state "
-                 << executor->state;
-      break;
   }
 }