Properly throw UnexpectedValueException for unconnected graphs

clue · clue · commit a392211d24ba · 2013-08-05T22:57:53.000+02:00
diff --git a/src/MinimumSpanningTree/Prim.php b/src/MinimumSpanningTree/Prim.php
@@ -56,8 +56,7 @@ public function getEdges()
                     $cheapestEdge = $edgeQueue->extract();
                     /* @var $cheapestEdge EdgeDirected */
                 } catch (Exception $e) {
-                    return $returnEdges;
-                    throw new UnexpectedValueException('Graph has more than one component');
+                    throw new UnexpectedValueException('Graph has more than one component', 0, $e);
                 }
 
                 // Check if edge is between unmarked and marked edge

Original file line number	Diff line number	Diff line change
`@@ -56,8 +56,7 @@ public function getEdges()`
`56`	`56`	`$cheapestEdge = $edgeQueue->extract();`
`57`	`57`	`/* @var $cheapestEdge EdgeDirected */`
`58`	`58`	`} catch (Exception $e) {`
`59`		`- return $returnEdges;`
`60`		`- throw new UnexpectedValueException('Graph has more than one component');`
	`59`	`+ throw new UnexpectedValueException('Graph has more than one component', 0, $e);`
`61`	`60`	`}`
`62`	`61`
`63`	`62`	`// Check if edge is between unmarked and marked edge`