[Bf-blender-cvs] [8dd9bb17498] master: Cleanup: avoid risky list as default argument in keymap
Campbell Barton
noreply at git.blender.org
Sun Mar 14 13:32:52 CET 2021
Commit: 8dd9bb17498148c543d932a2593ec22ffbbf85aa
Author: Campbell Barton
Date: Sun Mar 14 18:00:17 2021 +1100
Branches: master
https://developer.blender.org/rB8dd9bb17498148c543d932a2593ec22ffbbf85aa
Cleanup: avoid risky list as default argument in keymap
===================================================================
M release/scripts/presets/keyconfig/keymap_data/blender_default.py
M release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py
===================================================================
diff --git a/release/scripts/presets/keyconfig/keymap_data/blender_default.py b/release/scripts/presets/keyconfig/keymap_data/blender_default.py
index cf6a8152cff..19d5ef57cac 100644
--- a/release/scripts/presets/keyconfig/keymap_data/blender_default.py
+++ b/release/scripts/presets/keyconfig/keymap_data/blender_default.py
@@ -352,12 +352,12 @@ def _template_items_tool_select_actions(operator, *, type, value):
# This could have a more generic name, for now use for circle select.
-def _template_items_tool_select_actions_simple(operator, *, type, value, properties=[]):
+def _template_items_tool_select_actions_simple(operator, *, type, value, properties=()):
kmi_args = {"type": type, "value": value}
return [
# Don't define 'SET' here, take from the tool options.
(operator, kmi_args,
- {"properties": properties}),
+ {"properties": [*properties]}),
(operator, {**kmi_args, "shift": True},
{"properties": [*properties, ("mode", 'ADD')]}),
(operator, {**kmi_args, "ctrl": True},
diff --git a/release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py b/release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py
index 92f9be47de4..c8fc6abe10a 100644
--- a/release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py
+++ b/release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py
@@ -161,12 +161,12 @@ def _template_items_tool_select_actions(operator, *, type, value):
# This could have a more generic name, for now use for circle select.
-def _template_items_tool_select_actions_simple(operator, *, type, value, properties=[]):
+def _template_items_tool_select_actions_simple(operator, *, type, value, properties=()):
kmi_args = {"type": type, "value": value}
return [
# Don't define 'SET' here, take from the tool options.
(operator, kmi_args,
- {"properties": properties}),
+ {"properties": [*properties]}),
(operator, {**kmi_args, "shift": True},
{"properties": [*properties, ("mode", 'ADD')]}),
(operator, {**kmi_args, "ctrl": True},
More information about the Bf-blender-cvs
mailing list