From 53d2a9ca1a850f1d19a64a466e63eb9dd4c44283 Mon Sep 17 00:00:00 2001
From: Jan Petykiewicz <anewusername@gmail.com>
Date: Sat, 23 May 2020 19:39:48 -0700
Subject: [PATCH] Only swap between tuple/list if actually necessary

---
 masque/pattern.py | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/masque/pattern.py b/masque/pattern.py
index e3f8c8c..7599e06 100644
--- a/masque/pattern.py
+++ b/masque/pattern.py
@@ -764,10 +764,11 @@ class Pattern:
         Returns:
             self
         """
-        self.shapes = tuple(self.shapes)
-        self.labels = tuple(self.labels)
-        self.subpatterns = tuple(self.subpatterns)
-        object.__setattr__(self, 'locked', True)
+        if not self.locked:
+            self.shapes = tuple(self.shapes)
+            self.labels = tuple(self.labels)
+            self.subpatterns = tuple(self.subpatterns)
+            object.__setattr__(self, 'locked', True)
         return self
 
     def unlock(self) -> 'Pattern':
@@ -777,10 +778,11 @@ class Pattern:
         Returns:
             self
         """
-        object.__setattr__(self, 'locked', False)
-        self.shapes = list(self.shapes)
-        self.labels = list(self.labels)
-        self.subpatterns = list(self.subpatterns)
+        if self.locked:
+            object.__setattr__(self, 'locked', False)
+            self.shapes = list(self.shapes)
+            self.labels = list(self.labels)
+            self.subpatterns = list(self.subpatterns)
         return self
 
     def deeplock(self) -> 'Pattern':