From 8c8996a360187f5ae831b0bd97fc2589fcf2ff53 Mon Sep 17 00:00:00 2001 From: Raju Gupta Date: Wed, 17 May 2023 02:57:46 +0200 Subject: [PATCH] Update copy.py I think that some code was being repeated because of an if else condition. I tried to combine them into one. Would loved to know the thoughts of the community. --- Lib/copy.py | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/Lib/copy.py b/Lib/copy.py index da2908ef623d8c..8136d1818f5fa2 100644 --- a/Lib/copy.py +++ b/Lib/copy.py @@ -271,22 +271,16 @@ def _reconstruct(x, memo, func, args, setattr(y, key, value) if listiter is not None: - if deep: - for item in listiter: + for item in listiter: + if deep: item = deepcopy(item, memo) - y.append(item) - else: - for item in listiter: - y.append(item) + y.append(item) if dictiter is not None: - if deep: - for key, value in dictiter: + for key, value in dictiter: + if deep: key = deepcopy(key, memo) value = deepcopy(value, memo) - y[key] = value - else: - for key, value in dictiter: - y[key] = value + y[key] = value return y del types, weakref