gomacro icon indicating copy to clipboard operation
gomacro copied to clipboard

Convert to string using rune() again

Open QuLogic opened this issue 2 years ago • 2 comments

Not sure why this wasn't causing issues at the time of #101.

Also, I think the bounds check isn't necessary in this case because the Int64Val would have already done it, but I'm not certain.

QuLogic avatar Jan 30 '22 08:01 QuLogic

The fragment you propose to modify is supposed to perform untyped integer -> untyped string conversion.

It must return a string, not a rune

cosmos72 avatar Jan 30 '22 20:01 cosmos72

Oops, I meant to wrap that in string again, sorry.

QuLogic avatar Jan 31 '22 07:01 QuLogic