tinygo icon indicating copy to clipboard operation
tinygo copied to clipboard

Checking for methodset existance

Open marco6 opened this issue 1 year ago • 1 comments

This PR adds a check for the existence of the methodset in the interpreter implementation of typeasserts. This is necessary since this field is optional.

This should be the last bit that fixes #3788 since pull #3986 only covers XXX.(any) cases and still crashes if the interface has methods. As such, it should also fix #4079.

As a last note, I also tried a different approach here where I forced the methodset to be mandatory as I felt it was more robust, but the simplicity of these 4 lines seems to be unmatched.

marco6 avatar Feb 13 '24 16:02 marco6

/cc @aykevl

dgryski avatar Mar 05 '24 14:03 dgryski

Any progress on this? I'm also having this issue

TheSignPainter98 avatar May 08 '24 14:05 TheSignPainter98