TIC-80 icon indicating copy to clipboard operation
TIC-80 copied to clipboard

Wren textri broken

Open andraaspar opened this issue 3 years ago • 5 comments

Hi,

I am getting:

-1, "TIC metaclass does not implement 
'textri(_,_,_,_,_,_,_,_,_,_,_,_,_,_)'."

A regression since TIC-80 0.90.1723. You can see it on: Carcassonne when you start a new game with 2 players. (New game, click 2 colors, click tick on bottom.)

Edit: regression info.

andraaspar avatar Jun 05 '22 14:06 andraaspar

I'm not sure I understand why this workaround was necessary?

https://github.com/nesbox/TIC-80/commit/1f28e3aa86edb52181c75290782a2770965aaf7a

joshgoebel avatar Dec 04 '22 18:12 joshgoebel

we accidentally removed textri() in wren, should be restored

nesbox avatar Jan 12 '23 21:01 nesbox

restored textri() in Wren 5a47a828d82d38ca666f0a0c20f1d947b471d921

nesbox avatar Feb 06 '23 11:02 nesbox

It looks like textri was also removed from Fennel and Lua:

>unknown:14:7 Compile error: unknown identifier: textri

(trace >>textri<<)

This breaks Spilljackers on 1.2.

technomancy avatar Jun 14 '24 16:06 technomancy

I checked this using the game linked above, and textri is not working correctly in wren. You can see it when rotating tiles. It does not match the old version. It results in missing / broken texture.

On Sat, Jun 22, 2024, 08:38 Vadim Grigoruk @.***> wrote:

Reopened #1948 https://github.com/nesbox/TIC-80/issues/1948.

— Reply to this email directly, view it on GitHub https://github.com/nesbox/TIC-80/issues/1948#event-13249542503, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2ZQPSLLRKFC4Q52U5RUK3ZIULXNAVCNFSM6AAAAABJKTVESCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGI2DSNJUGI2TAMY . You are receiving this because you authored the thread.Message ID: @.***>

andraaspar avatar Jun 22 '24 06:06 andraaspar