jyapayne/vimspector
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix LGTM error: python default arguments are mutable\!

Browse source
This commit is contained in:
Ben Jackson 2020-02-16 17:50:28 +00:00
commit 267f202dad
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
python3/vimspector/debug_session.py
View file

@ -72,7 +72,12 @@ class DebugSession( object ):
self._on_init_complete_handlers = [] self._on_init_complete_handlers = []
self._server_capabilities = {} self._server_capabilities = {}
def Start( self, launch_variables = {} ): def Start( self, launch_variables = None ):
# We mutate launch_variables, so don't mutate the default argument.
# https://docs.python-guide.org/writing/gotchas/#mutable-default-arguments
if launch_variables is None:
launch_variables = {}
self._logger.info( "User requested start debug session with %s", self._logger.info( "User requested start debug session with %s",
launch_variables ) launch_variables )
self._configuration = None self._configuration = None