Skip to content
Snippets Groups Projects
Commit c32ec336 authored by Maxence Naud's avatar Maxence Naud
Browse files

Fix Node::operator() that wrongly checked each input instead of data only

parent 4b2dc107
No related branches found
No related tags found
No related merge requests found
......@@ -39,11 +39,15 @@ Aidge::Node::Node(std::shared_ptr<Operator> op, const std::string& name)
Aidge::Connector Aidge::Node::operator()(const std::vector<Connector>& ctors) {
assert((ctors.size() == nbData()) && "Wrong number of arguments.\n");
for (std::pair<std::shared_ptr<Node>, IOIndex_t>& input : inputs()) {
assert((gk_IODefaultIndex == input.second) &&
for (std::size_t i = 0; i < nbData(); i++) {
assert((gk_IODefaultIndex == input(i).second) &&
"At least one input connection is not free.\n");
(void)input; // avoid unused warning
}
// for (std::pair<std::shared_ptr<Node>, IOIndex_t>& input : inputs()) {
// assert((gk_IODefaultIndex == input.second) &&
// "At least one input connection is not free.\n");
// (void)input; // avoid unused warning
// }
IOIndex_t i = 0;
for (const Connector& ctor : ctors) {
if (ctor.node() != nullptr) { // ctor must be associated with a node
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment