Skip to content

Commit 8601c1c

Browse files
authored
fix: avoid side effects in PdlContext (#1054)
Signed-off-by: Louis Mandel <[email protected]>
1 parent fb308a9 commit 8601c1c

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

src/pdl/pdl_context.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -68,16 +68,16 @@ def __init__(self, context: list[PDLContext | dict[str, Any]]):
6868
for item in context:
6969
match item:
7070
case IndependentContext():
71-
ret += item.context.data
71+
ret = ret + item.context.data
7272
case SingletonContext():
73-
ret += [item]
73+
ret = ret + [item]
7474
case DependentContext():
7575
if len(item) == 0:
7676
pass
7777
else:
78-
ret += [item]
78+
ret = ret + [item]
7979
case dict():
80-
ret += [SingletonContext(item)]
80+
ret = ret + [SingletonContext(item)]
8181
case _:
8282
assert False
8383
self.context = PdlList(ret)
@@ -98,7 +98,7 @@ def __getitem__(self, index: int | slice): # pyright: ignore
9898

9999
def __repr__(self): # pyright: ignore
100100
ret = "{"
101-
ret += ",".join([i.__repr__() for i in self.context.result()])
101+
ret = ret + ",".join([i.__repr__() for i in self.context.result()])
102102
return ret + "}"
103103

104104

@@ -110,16 +110,16 @@ def __init__(self, context: list[PDLContext | dict[str, Any]]):
110110
for item in context:
111111
match item:
112112
case DependentContext():
113-
ret += item.context.data
113+
ret = ret + item.context.data
114114
case SingletonContext():
115-
ret += [item]
115+
ret = ret + [item]
116116
case IndependentContext():
117117
if len(item) == 0:
118118
pass
119119
else:
120-
ret += [item]
120+
ret = ret + [item]
121121
case dict():
122-
ret += [SingletonContext(item)]
122+
ret = ret + [SingletonContext(item)]
123123
case _:
124124
assert False
125125
self.context = PdlList(ret)
@@ -138,7 +138,7 @@ def __getitem__(self, index: int | slice): # pyright: ignore
138138

139139
def __repr__(self): # pyright: ignore
140140
ret = "["
141-
ret += ",".join([i.__repr__() for i in self.context.result()])
141+
ret = ret + ",".join([i.__repr__() for i in self.context.result()])
142142
return ret + "]"
143143

144144

0 commit comments

Comments
 (0)