Fix #125: Modify pdigraph for consistent behaviour

POETSII / Orchestrator

The Orchestrator is the configuration and run-time management system for POETS platforms.

1 stars 1 forks source link

Fix #125: Modify pdigraph for consistent behaviour #127

Closed heliosfa closed 4 years ago

heliosfa commented 4 years ago

This PR changes the order of operations in _PDIGRAPH::FindArcs on line 255 of pdigraph.tpp so that the arrays are cleared before checking if the pin/node exist. This is consistent with _PDIGRAPH::FindArcs on line 279 and gives a more expected and correct behaviour.

Discussed with @mvousden and ADB

heliosfa commented 4 years ago

All tests pass on the branch being merged. Just waiting for the memory checks to run (could take a while)