Details
Trivial Description
In the course of fixing MESOS-947, it has been overlooked by me that the code in question became dead code. Coverty caught this and Ben Mahler reported it.
At the top of _runTask(), there is now a test whether framework is NULL and in each case it ever is before reaching the code in question (see below) there is a local exit from the method. So we should be able to remove this code without effect:
- if (framework == NULL)
{
- framework = new Framework(this, frameworkId, frameworkInfo, pid);
- frameworks[frameworkId] = framework;
- }
-
- CHECK_NOTNULL(framework);
-
