From bb6b87a2685c0d71b5c38be33c100f85ac0b9cee Mon Sep 17 00:00:00 2001
From: Renaud G <renaud@rolisteam.org>
Date: Thu, 25 Jul 2019 09:59:29 +0200
Subject: Rework of the component to be a proper lib

---
 node/rerolldicenode.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'node/rerolldicenode.cpp')

diff --git a/node/rerolldicenode.cpp b/node/rerolldicenode.cpp
index 56ee6d9..18e8bba 100644
--- a/node/rerolldicenode.cpp
+++ b/node/rerolldicenode.cpp
@@ -79,7 +79,8 @@ void RerollDiceNode::run(ExecutionNode* previous)
         }
         else
         {
-            m_errors.insert(ExecutionNode::DIE_RESULT_EXPECTED,
+            m_errors.insert(
+                Dice::ERROR_CODE::DIE_RESULT_EXPECTED,
                 QObject::tr(
                     " The a operator expects dice result. Please check the documentation and fix your command."));
         }
-- 
cgit v1.2.3-70-g09d2


From a092fb4e6931d4af233b02fc3bd08c59681b03cb Mon Sep 17 00:00:00 2001
From: Renaud G <renaud@rolisteam.org>
Date: Sun, 28 Jul 2019 02:36:36 +0200
Subject: manage new API isValidRangeSize

---
 node/rerolldicenode.cpp | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

(limited to 'node/rerolldicenode.cpp')

diff --git a/node/rerolldicenode.cpp b/node/rerolldicenode.cpp
index 18e8bba..ee00e00 100644
--- a/node/rerolldicenode.cpp
+++ b/node/rerolldicenode.cpp
@@ -1,5 +1,6 @@
 #include "rerolldicenode.h"
 #include "parsingtoolbox.h"
+#include <utility>
 
 RerollDiceNode::RerollDiceNode(bool reroll, bool addingMode)
     : m_diceResult(new DiceResult()), m_validator(nullptr), m_reroll(reroll), m_adding(addingMode)
@@ -26,7 +27,6 @@ void RerollDiceNode::run(ExecutionNode* previous)
             for(auto& die : previous_result->getResultList())
             {
                 Die* tmpdie= new Die(*die);
-                //*tmpdie= *die;
                 m_diceResult->insertResult(tmpdie);
                 die->displayed();
             }
@@ -35,10 +35,22 @@ void RerollDiceNode::run(ExecutionNode* previous)
             QList<Die*>& list= m_diceResult->getResultList();
             QList<Die*> toRemove;
 
-            for(int i= 0; i < list.size(); ++i)
+            for(auto& die : list)
             {
-                auto die= list.at(i);
                 bool finished= false;
+                auto state
+                    = m_validator->isValidRangeSize(std::make_pair<qint64, qint64>(die->getBase(), die->getMaxValue()));
+                if((Dice::CONDITION_STATE::ALWAYSTRUE == state && m_adding)
+                   || (!m_reroll && !m_adding && state == Dice::CONDITION_STATE::UNREACHABLE))
+                {
+                    m_errors.insert(Dice::ERROR_CODE::ENDLESS_LOOP_ERROR,
+                                    QObject::tr("Condition (%1) cause an endless loop with this dice: %2")
+                                        .arg(toString(true))
+                                        .arg(QStringLiteral("d[%1,%2]")
+                                                 .arg(static_cast<int>(die->getBase()))
+                                                 .arg(static_cast<int>(die->getMaxValue()))));
+                    continue;
+                }
                 while(m_validator->hasValid(die, false) && !finished)
                 {
                     if(m_instruction != nullptr)
-- 
cgit v1.2.3-70-g09d2


From a31826330e85c4c3694e85319da863b6d1784257 Mon Sep 17 00:00:00 2001
From: Renaud G <renaud@rolisteam.org>
Date: Sun, 28 Jul 2019 02:37:12 +0200
Subject: prevent crash

---
 node/rerolldicenode.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'node/rerolldicenode.cpp')

diff --git a/node/rerolldicenode.cpp b/node/rerolldicenode.cpp
index ee00e00..70475a5 100644
--- a/node/rerolldicenode.cpp
+++ b/node/rerolldicenode.cpp
@@ -64,8 +64,8 @@ void RerollDiceNode::run(ExecutionNode* previous)
                             {
                                 toRemove.append(die);
                                 list.append(lastResult->getResultList());
+                                lastResult->clear();
                             }
-                            lastResult->clear();
                         }
                     }
                     else
-- 
cgit v1.2.3-70-g09d2