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;
}
}