error.Error() does not work

[pierrec]

Error:

   15 |     so_panic(so_cstr(fmt_Sprintf(buf, "%s: %s", so_cstr(err.Error(err.self)), so_cstr(msg))));
      |                                                         ~~~^

with:

func boom(err error, msg string) {
	buf := fmt.NewBuffer(1 << 10)
	panic(fmt.Sprintf(buf, "%s: %s", err.Error(), msg))
}

[nalgeon]

The error.Error method is not supported, I'm afraid. You can just pass the error object itself:

panic(fmt.Sprintf(buf, "%s: %s", err, msg))

[pierrec]

The error.Error method is not supported, I'm afraid. You can just pass the error object itself:

panic(fmt.Sprintf(buf, "%s: %s", err, msg))

This works indeed. I only tried to use err.String() as I am not used to using C verbs instead of Go's and it was not obvious to me that passing err with %s would work.

However, before your reply, Claude made a patch for this that fixed the issue. It pretty straight forward, if interested, otherwise I'll close this.

[nalgeon]

I can't promise that I'll merge the patch, but I'd definitely like to look at it.

[nalgeon]

The PR has been merged, so I'm closing the issue. Thank you for your contribution!