Fix/issue 228 (#230)

* fix: issue 221

The return control flow was not being processed in two of the dynamic array types.

* fixes to deal with nested arrays.

Other patterns were tested and it all seems to work.

Author: paxcut

lib/source/pl/core/ast/ast_node_array_variable_decl.cpp

@@ -334,11 +334,13 @@ namespace pl::core::ast {
                         addEntries(std::move(patterns));
 
                     auto ctrlFlow = evaluator->getCurrentControlFlowStatement();
-                    evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
-                    if (ctrlFlow == ControlFlowStatement::Break || ctrlFlow == ControlFlowStatement::Return)
+                    if ( ctrlFlow == ControlFlowStatement::Return) {
                         break;
-                    else if (ctrlFlow == ControlFlowStatement::Continue) {
-
+                    } else if (ctrlFlow == ControlFlowStatement::Break) {
+                        evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
+                        break;
+                    } else if (ctrlFlow == ControlFlowStatement::Continue) {
+                        evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
                         discardEntries(patternCount);
                         continue;
                     }
@@ -366,10 +368,13 @@ namespace pl::core::ast {
 
 
                     auto ctrlFlow = evaluator->getCurrentControlFlowStatement();
-                    evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
-                    if (ctrlFlow == ControlFlowStatement::Break || ctrlFlow == ControlFlowStatement::Return)
+                    if ( ctrlFlow == ControlFlowStatement::Return) {
+                        break;
+                    } else if (ctrlFlow == ControlFlowStatement::Break) {
+                        evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
                         break;
-                    else if (ctrlFlow == ControlFlowStatement::Continue) {
+                    } else if (ctrlFlow == ControlFlowStatement::Continue) {
+                        evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
                         discardEntries(patternCount);
                         continue;
                     }
@@ -417,10 +422,13 @@ namespace pl::core::ast {
                 }
 
                 auto ctrlFlow = evaluator->getCurrentControlFlowStatement();
-                evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
-                if (ctrlFlow == ControlFlowStatement::Break || ctrlFlow == ControlFlowStatement::Return)
+                if ( ctrlFlow == ControlFlowStatement::Return) {
+                    break;
+                } else if (ctrlFlow == ControlFlowStatement::Break) {
+                    evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
                     break;
-                else if (ctrlFlow == ControlFlowStatement::Continue) {
+                } else if (ctrlFlow == ControlFlowStatement::Continue) {
+                    evaluator->setCurrentControlFlowStatement(ControlFlowStatement::None);
                     discardEntries(1);
                     continue;
                 }