BUGFIX: Fix some more bugs resulting from const cleanup

ArcaneNibble · ArcaneNibble · commit 2e8f215dcafc · 2017-06-10T00:02:20.000-07:00
Also added the "override" keyword to prevent this happening again
diff --git a/src/gp4par/Greenpak4PAREngine.cpp b/src/gp4par/Greenpak4PAREngine.cpp
@@ -209,7 +209,7 @@ uint32_t Greenpak4PAREngine::ComputeCongestionCost() const
 ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 // Print logic
 
-void Greenpak4PAREngine::PrintUnroutes(vector<PARGraphEdge*>& unroutes)
+void Greenpak4PAREngine::PrintUnroutes(vector<const PARGraphEdge*>& unroutes) const
 {
 	Log(Severity::ERROR, "\nUnroutable nets (%zu):\n", unroutes.size());
 	LogIndenter li;
@@ -364,7 +364,7 @@ bool Greenpak4PAREngine::CantMoveSrc(Greenpak4BitstreamEntity* src)
 	return false;
 }
 
-bool Greenpak4PAREngine::CanMoveNode(PARGraphNode* node, PARGraphNode* old_mate, PARGraphNode* new_mate)
+bool Greenpak4PAREngine::CanMoveNode(PARGraphNode* node, PARGraphNode* old_mate, PARGraphNode* new_mate) const
 {
 	//Let base class filter stuff
 	if(!PAREngine::CanMoveNode(node, old_mate, new_mate))
diff --git a/src/gp4par/Greenpak4PAREngine.h b/src/gp4par/Greenpak4PAREngine.h
@@ -29,15 +29,15 @@ class Greenpak4PAREngine : public PAREngine
 	virtual ~Greenpak4PAREngine();
 
 protected:
-	virtual void PrintUnroutes(std::vector<PARGraphEdge*>& unroutes);
+	virtual void PrintUnroutes(std::vector<const PARGraphEdge*>& unroutes) const override;
 
-	virtual void FindSubOptimalPlacements(std::vector<PARGraphNode*>& bad_nodes);
-	virtual PARGraphNode* GetNewPlacementForNode(PARGraphNode* pivot);
+	virtual void FindSubOptimalPlacements(std::vector<PARGraphNode*>& bad_nodes) override;
+	virtual PARGraphNode* GetNewPlacementForNode(PARGraphNode* pivot) override;
 
-	virtual uint32_t ComputeCongestionCost() const;
-	virtual bool InitialPlacement_core();
+	virtual uint32_t ComputeCongestionCost() const override;
+	virtual bool InitialPlacement_core() override;
 
-	virtual bool CanMoveNode(PARGraphNode* node, PARGraphNode* old_mate, PARGraphNode* new_mate);
+	virtual bool CanMoveNode(PARGraphNode* node, PARGraphNode* old_mate, PARGraphNode* new_mate) const override;
 
 	bool CantMoveSrc(Greenpak4BitstreamEntity* src);
 	bool CantMoveDst(Greenpak4BitstreamEntity* dst);

Original file line number	Diff line number	Diff line change
`@@ -209,7 +209,7 @@ uint32_t Greenpak4PAREngine::ComputeCongestionCost() const`
`209`	`209`	`////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////`
`210`	`210`	`// Print logic`
`211`	`211`
`212`		`-void Greenpak4PAREngine::PrintUnroutes(vector<PARGraphEdge*>& unroutes)`
	`212`	`+void Greenpak4PAREngine::PrintUnroutes(vector<const PARGraphEdge*>& unroutes) const`
`213`	`213`	`{`
`214`	`214`	`Log(Severity::ERROR, "\nUnroutable nets (%zu):\n", unroutes.size());`
`215`	`215`	`LogIndenter li;`
`@@ -364,7 +364,7 @@ bool Greenpak4PAREngine::CantMoveSrc(Greenpak4BitstreamEntity* src)`
`364`	`364`	`return false;`
`365`	`365`	`}`
`366`	`366`
`367`		`-bool Greenpak4PAREngine::CanMoveNode(PARGraphNode* node, PARGraphNode* old_mate, PARGraphNode* new_mate)`
	`367`	`+bool Greenpak4PAREngine::CanMoveNode(PARGraphNode* node, PARGraphNode* old_mate, PARGraphNode* new_mate) const`
`368`	`368`	`{`
`369`	`369`	`//Let base class filter stuff`
`370`	`370`	`if(!PAREngine::CanMoveNode(node, old_mate, new_mate))`