Infinite Loop in ClearCalculation Function about node_editor_framework HOT 4 CLOSED

this fixed problem for me (NodeEditor.cs:848):

if (!node.allowRecursion) ClearCalculation (output.connections [conCnt].body);

from node_editor_framework.

Thanks for the report, I initially handled such problems with the node.shouldCalculate bool (like in the AllAroundNode Test node). The case allowRecursion = true and shouldCalculate = true was thought for the node itself to handle in the Calculation function. Though it seems I missed to handle that case in ClearCalculation:( Recursion is a bit tricky, and I should definitely rework how it's handled to prevent such crashed...

from node_editor_framework.

My nodes already have allowRecursion and the reason I cant use shouldCalculate is that I need my nodes to calculate and update values with any connection change.
Thanks for your respond.

from node_editor_framework.

My latest commit (2dd6eb8) adresses this and some minor othr problem, be sure to check it out;) The Recursive system is now totally secure, even if you chose allowRecursion = true and shouldCalculate = true

from node_editor_framework.