Remove unreachable code in basic_consolidation.py

In basic_consolidation.py there is a method called calculate_score_vm()
which had two return statements following each other. The second one
(which equals the first one) is unreachable as the first one returns.

Change-Id: Ia4877c22188fae6217e07597a2dd939633414349
Closes-Bug: #1524911

watcher/decision_engine/strategy/basic_consolidation.py

@@ -321,10 +321,6 @@ class BasicConsolidation(BaseStrategy):
                                      0,
                                      0)
 
-        return self.calculate_weight(model, vm, total_cores_used,
-                                     0,
-                                     0)
-
     def print_utilization(self, model):
         if model is None:
             raise ClusteStateNotDefined()