Make Project.addTestRunnerConfiguration() accept a const GeneratorSet…

…tings argument
dlang · Jul 14, 2022 · 281ee32 · 281ee32
1 parent adb907d
commit 281ee32
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/source/dub/generators/build.d b/source/dub/generators/build.d
@@ -32,7 +32,7 @@ string getObjSuffix(const scope ref BuildPlatform platform)
     return platform.isWindows() ? ".obj" : ".o";
 }
 
-string computeBuildName(string config, GeneratorSettings settings, const string[][] hashing...)
+string computeBuildName(string config, in GeneratorSettings settings, const string[][] hashing...)
 {
 	import std.digest;
 	import std.digest.md;

diff --git a/source/dub/project.d b/source/dub/project.d
@@ -231,7 +231,7 @@ class Project {
 		Returns:
 			Name of the added test runner configuration, or null for base configurations with target type `none`
 	*/
-	string addTestRunnerConfiguration(GeneratorSettings settings, bool generate_main = true, string base_config = "", NativePath custom_main_file = NativePath())
+	string addTestRunnerConfiguration(in GeneratorSettings settings, bool generate_main = true, string base_config = "", NativePath custom_main_file = NativePath())
 	{
 		if (base_config.length == 0) {
 			// if a custom main file was given, favor the first library configuration, so that it can be applied
@@ -244,7 +244,7 @@ class Project {
 			if (!base_config.length) base_config = getDefaultConfiguration(settings.platform, true);
 		}
 
-		BuildSettings lbuildsettings = settings.buildSettings;
+		BuildSettings lbuildsettings = settings.buildSettings.dup;
 		addBuildSettings(lbuildsettings, settings, base_config, null, true);
 
 		if (lbuildsettings.targetType == TargetType.none) {