From 502a684872d6ffd80f31c61ce67e5d856b472da2 Mon Sep 17 00:00:00 2001 From: Dylan Baker Date: Tue, 14 May 2019 16:36:10 -0700 Subject: build: TestSetup doesn't take keyword arguments This function is currently setup with keyword arguments defaulting to None. However, it is never called without passing all of it's arguments explicitly, and only one of it's arguments would actually be valid as None. So just drop that, and make them all positional. And annotate them. --- mesonbuild/interpreter.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'mesonbuild/interpreter.py') diff --git a/mesonbuild/interpreter.py b/mesonbuild/interpreter.py index d4c454c..e2b9631 100644 --- a/mesonbuild/interpreter.py +++ b/mesonbuild/interpreter.py @@ -3765,10 +3765,7 @@ different subdirectory. 'is_default can be set to true only once' % self.build.test_setup_default_name) self.build.test_setup_default_name = setup_name env = self.unpack_env_kwarg(kwargs) - self.build.test_setups[setup_name] = build.TestSetup(exe_wrapper=exe_wrapper, - gdb=gdb, - timeout_multiplier=timeout_multiplier, - env=env) + self.build.test_setups[setup_name] = build.TestSetup(exe_wrapper, gdb, timeout_multiplier, env) def get_argdict_on_crossness(self, native_dict, cross_dict, kwargs): for_native = kwargs.get('native', not self.environment.is_cross_build()) -- cgit v1.1