From 9cc4518087c6f1de1be39de22e5b63c7849a145d Mon Sep 17 00:00:00 2001
From: Emmanuel Garette <egarette@cadoles.com>
Date: Mon, 2 Dec 2019 10:42:17 +0100
Subject: [PATCH] copy value before store it

---
 tiramisu/value.py | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/tiramisu/value.py b/tiramisu/value.py
index 9af8d14..f43db13 100644
--- a/tiramisu/value.py
+++ b/tiramisu/value.py
@@ -90,7 +90,8 @@ class Values(object):
                                             check_error=False)
         if isinstance(value, list):
             # return a copy, so value cannot be modified
-            return value.copy()
+            from copy import copy
+            value = copy(value)
         # and return it
         return value
 
@@ -177,12 +178,8 @@ class Values(object):
                 return value
 
         # now try to get default value:
-        value = option_bag.option.impl_getdefault()
-
-        # - if option is a submulti, return a list a list
-        # - if option is a multi, return a list
-        # - default value
-        value = self.calc_value(option_bag, value)
+        value = self.calc_value(option_bag,
+                                option_bag.option.impl_getdefault())
         if option_bag.option.impl_is_multi() and option_bag.index is not None and isinstance(value, (list, tuple)):
             # if index, must return good value for this index
             if len(value) > option_bag.index:
@@ -190,7 +187,8 @@ class Values(object):
             else:
                 # no value for this index, retrieve default multi value
                 # default_multi is already a list for submulti
-                value = self.calc_value(option_bag, option_bag.option.impl_getdefault_multi())
+                value = self.calc_value(option_bag,
+                                        option_bag.option.impl_getdefault_multi())
         return value
 
     def calculate_reset_cache(self, option_bag, value):
@@ -285,6 +283,9 @@ class Values(object):
             self.setvalue_validation(value,
                                      option_bag)
 
+        if isinstance(value, list):
+            # copy
+            value = value.copy()
         self._setvalue(option_bag,
                        value,
                        owner,
@@ -334,9 +335,6 @@ class Values(object):
                   owner,
                   commit=True):
         option_bag.config_bag.context.cfgimpl_reset_cache(option_bag)
-        if isinstance(value, list):
-            # copy
-            value = value.copy()
         self._p_.setvalue(option_bag.path,
                           value,
                           owner,