builtin,cgen: add a source .location field to $for method in Type.methods { (#25976)

spytheman · web-flow · commit dec703006ba0 · 2025-12-15T14:38:48.000+02:00
diff --git a/vlib/builtin/meta_function.v b/vlib/builtin/meta_function.v
@@ -11,6 +11,7 @@ pub:
 pub struct FunctionData {
 pub:
 	name        string
+	location    string
 	attrs       []string
 	args        []FunctionParam
 	return_type int
diff --git a/vlib/v/gen/c/comptime.v b/vlib/v/gen/c/comptime.v
@@ -662,6 +662,8 @@ fn (mut g Gen) comptime_for(node ast.ComptimeFor) {
 			g.comptime.comptime_for_method_var = node.val_var
 			g.writeln('/* method ${i} : ${method.name} */ {')
 			g.writeln('\t${node.val_var}.name = _S("${method.name}");')
+			mlocation := util.cescaped_path(util.path_styled_for_error_messages(method.file))
+			g.writeln('\t${node.val_var}.location = _S("${mlocation}:${method.name_pos.line_nr + 1}:${method.name_pos.col}");')
 			if method.attrs.len == 0 {
 				g.writeln('\t${node.val_var}.attrs = builtin____new_array_with_default(0, 0, sizeof(string), 0);')
 			} else {
diff --git a/vlib/v/gen/c/testdata/attr_string_quotes_escape.out b/vlib/v/gen/c/testdata/attr_string_quotes_escape.out
@@ -1,7 +1 @@
-FunctionData{
-    name: 'sample_method'
-    attrs: ['option: "1"']
-    args: []
-    return_type: 1
-    typ: 0
-}
+['option: "1"']
diff --git a/vlib/v/gen/c/testdata/attr_string_quotes_escape.vv b/vlib/v/gen/c/testdata/attr_string_quotes_escape.vv
@@ -6,6 +6,6 @@ fn (d Dummy) sample_method() {
 
 fn main() {
 	$for method in Dummy.methods {
-		println(method)
+		println(method.attrs)
 	}
 }
diff --git a/vlib/v/tests/comptime/comptime_call_test.v b/vlib/v/tests/comptime/comptime_call_test.v
@@ -36,16 +36,23 @@ fn test_for_methods() {
 		$if method.return_type is string {
 			v := test.$method()
 			r += v.str()
+			if method.name == 's' {
+				assert method.location.ends_with('vlib/v/tests/comptime/comptime_call_test.v:11:15')
+			} else if method.name == 's2' {
+				assert method.location.ends_with('vlib/v/tests/comptime/comptime_call_test.v:15:15')
+			}
 		} $else $if method.return_type is int {
 			// TODO
 			// v := test.$method()
 			v := '?'
 			r += v.str()
 			assert method.name == 'i'
+			assert method.location.ends_with('vlib/v/tests/comptime/comptime_call_test.v:7:15')
 		} $else {
 			// no return type
 			test.$method()
 			assert method.name == 'v'
+			assert method.location.ends_with('vlib/v/tests/comptime/comptime_call_test.v:3:15')
 		}
 	}
 	assert r == '?testTwo'

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,6 @@ fn (d Dummy) sample_method() {`
`6`	`6`
`7`	`7`	`fn main() {`
`8`	`8`	`$for method in Dummy.methods {`
`9`		`- println(method)`
	`9`	`+ println(method.attrs)`
`10`	`10`	`}`
`11`	`11`	`}`