link: Simplify library path resolution logic

Author: Doekin

src/link.zig

@@ -2136,29 +2136,43 @@ fn resolveLibInput(
 
     const lib_name = name_query.name;
 
-    if (target.os.tag.isDarwin() and link_mode == .dynamic) tbd: {
-        // Prefer .tbd over .dylib.
-        const test_path: Path = .{
-            .root_dir = lib_directory,
-            .sub_path = try std.fmt.allocPrint(arena, "lib{s}.tbd", .{lib_name}),
-        };
-        try checked_paths.writer(gpa).print("\n  {}", .{test_path});
-        var file = test_path.root_dir.handle.openFile(test_path.sub_path, .{}) catch |err| switch (err) {
-            error.FileNotFound => break :tbd,
-            else => |e| fatal("unable to search for tbd library '{}': {s}", .{ test_path, @errorName(e) }),
+    const LibPathPattern = struct {
+        prefix: []const u8,
+        suffix: []const u8,
+    };
+    const default_patterns = &[_]LibPathPattern{.{ .prefix = target.libPrefix(), .suffix = switch (link_mode) {
+        .static => target.staticLibSuffix(),
+        .dynamic => target.dynamicLibSuffix(),
+    } }};
+    var candidate_patterns: []const LibPathPattern = default_patterns;
+
+    if (target.os.tag.isDarwin() and link_mode == .dynamic) {
+        candidate_patterns =
+            // Prefer .tbd over .dylib.
+            [_]LibPathPattern{.{ .prefix = "lib", .suffix = ".tbd" }}
+            // In the case of Darwin, the main check will be .dylib
+            ++ default_patterns
+            // Additionally check for .so files.
+            ++ [_]LibPathPattern{.{ .prefix = "lib", .suffix = ".so" }};
+    } else if (target.isMinGW()) {
+        // In the case of MinGW, the main check will be `libfoo.dll` and `libfoo.a`, but we also need to
+        // look for `libfoo.dll.a`, `foo.dll` and `foo.lib`.
+        candidate_patterns = switch (link_mode) {
+            .dynamic => default_patterns ++ [_]LibPathPattern{
+                .{ .prefix = "lib", .suffix = ".dll.a" },
+                .{ .prefix = "", .suffix = ".dll" },
+            },
+            .static => default_patterns ++ [_]LibPathPattern{
+                .{ .prefix = "", .suffix = ".lib" },
+            },
         };
-        errdefer file.close();
-        return finishResolveLibInput(resolved_inputs, test_path, file, link_mode, name_query.query);
     }
 
-    {
+    for (candidate_patterns) |pattern| {
         const test_path: Path = .{
             .root_dir = lib_directory,
             .sub_path = try std.fmt.allocPrint(arena, "{s}{s}{s}", .{
-                target.libPrefix(), lib_name, switch (link_mode) {
-                    .static => target.staticLibSuffix(),
-                    .dynamic => target.dynamicLibSuffix(),
-                },
+                pattern.prefix, lib_name, pattern.suffix,
             }),
         };
         try checked_paths.writer(gpa).print("\n  {}", .{test_path});
@@ -2171,52 +2185,6 @@ fn resolveLibInput(
         }
     }
 
-    // In the case of Darwin, the main check will be .dylib, so here we
-    // additionally check for .so files.
-    if (target.os.tag.isDarwin() and link_mode == .dynamic) so: {
-        const test_path: Path = .{
-            .root_dir = lib_directory,
-            .sub_path = try std.fmt.allocPrint(arena, "lib{s}.so", .{lib_name}),
-        };
-        try checked_paths.writer(gpa).print("\n  {}", .{test_path});
-        var file = test_path.root_dir.handle.openFile(test_path.sub_path, .{}) catch |err| switch (err) {
-            error.FileNotFound => break :so,
-            else => |e| fatal("unable to search for so library '{}': {s}", .{
-                test_path, @errorName(e),
-            }),
-        };
-        errdefer file.close();
-        return finishResolveLibInput(resolved_inputs, test_path, file, link_mode, name_query.query);
-    }
-
-    // In the case of MinGW, the main check will be `libfoo.dll` and `libfoo.a`, but we also need to
-    // look for `foo.dll`, `foo.lib` and `libfoo.dll.a`.
-    if (target.isMinGW()) {
-        const sub_paths = if (link_mode == .dynamic)
-            &[_][]const u8{
-                try std.fmt.allocPrint(arena, "lib{s}.dll.a", .{lib_name}),
-                try std.fmt.allocPrint(arena, "{s}.dll", .{lib_name}),
-                try std.fmt.allocPrint(arena, "{s}.lib", .{lib_name}),
-            }
-        else
-            &[_][]const u8{
-                try std.fmt.allocPrint(arena, "{s}.lib", .{lib_name}),
-            };
-        for (sub_paths) |sub_path| {
-            const test_path: Path = .{
-                .root_dir = lib_directory,
-                .sub_path = sub_path,
-            };
-            try checked_paths.writer(gpa).print("\n  {}", .{test_path});
-            var file = test_path.root_dir.handle.openFile(test_path.sub_path, .{}) catch |err| switch (err) {
-                error.FileNotFound => continue,
-                else => |e| fatal("unable to search for {s} library '{}': {s}", .{ @tagName(link_mode), test_path, @errorName(e) }),
-            };
-            errdefer file.close();
-            return finishResolveLibInput(resolved_inputs, test_path, file, link_mode, name_query.query);
-        }
-    }
-
     return .no_match;
 }