Improve proc group resolution with named args

2 files changed, 49 insertions, 1 deletions

diff --git a/src/server/analysis.odin b/src/server/analysis.odin
index 8bc311a..ff94e41 100644
--- a/src/server/analysis.odin
+++ b/src/server/analysis.odin
@@ -729,7 +729,7 @@ resolve_function_overload :: proc(ast_context: ^AstContext, group: ast.Proc_Grou
 
 				if !resolve_all_possibilities {
 					arg_count := get_proc_arg_count(procedure)
-					if call_expr != nil && arg_count < call_unnamed_arg_count {
+					if call_expr != nil && arg_count < len(call_expr.args) {
 						break next_fn
 					}
 				}
diff --git a/tests/hover_test.odin b/tests/hover_test.odin
index 6461621..a90346e 100644
--- a/tests/hover_test.odin
+++ b/tests/hover_test.odin
@@ -5234,6 +5234,54 @@ ast_hover_proc_group_with_enum_arg :: proc(t: ^testing.T) {
 	}
 	test.expect_hover(t, &source, "test.bar :: proc(foo: Foo)")
 }
+
+@(test)
+ast_hover_proc_group_with_enum_named_arg :: proc(t: ^testing.T) {
+	source := test.Source {
+		main = `package test
+		Foo :: enum {
+			A,
+			B,
+			C,
+		}
+
+		bar_none :: proc() {}
+		bar_foo :: proc(i: int, foo: Foo) {}
+		bar :: proc {
+			bar_none,
+			bar_foo,
+		}
+
+		main :: proc() {
+			b{*}ar(foo = .B, i = 2)
+		}
+
+		`,
+	}
+	test.expect_hover(t, &source, "test.bar :: proc(i: int, foo: Foo)")
+}
+
+@(test)
+ast_hover_proc_group_named_arg_with_nil :: proc(t: ^testing.T) {
+	source := test.Source {
+		main = `package test
+		Foo :: struct {}
+
+		bar_none :: proc() {}
+		bar_foo :: proc(i: int, foo: ^Foo) {}
+		bar :: proc {
+			bar_none,
+			bar_foo,
+		}
+
+		main :: proc() {
+			b{*}ar(foo = nil, i = 2)
+		}
+
+		`,
+	}
+	test.expect_hover(t, &source, "test.bar :: proc(i: int, foo: ^Foo)")
+}
 /*
 
 Waiting for odin fix