Bundled lua with 5.7 crashes when you right click a farmer villager. math.random needs correct order #3576

ancientmarinerdev · 2023-03-22T01:15:50+01:00

ancientmarinerdev commented

2023-03-22 01:15:50 +01:00

When playing 5.7 with bundled Lua, right click on villager crashes when args are in the incorrect order. math.random for bundled lua seems to want args in ascending order. I have checked other trades and it seems just this 1 was out of order.

Testing

Changed order, and it no longer crashes.

When playing 5.7 with bundled Lua, right click on villager crashes when args are in the incorrect order. math.random for bundled lua seems to want args in ascending order. I have checked other trades and it seems just this 1 was out of order. ### Testing Changed order, and it no longer crashes.

👍 1

ancientmarinerdev added the

mobs

#P1 CRITICAL

labels 2023-03-22 01:16:03 +01:00

ancientmarinerdev added this to the 0.83.0 - Safe and Sound milestone 2023-03-22 01:16:11 +01:00

ancientmarinerdev force-pushed fix_villager_crash_with_bundle_lua from 371922a8f1 to 2440b73e67

2023-03-22 04:55:37 +01:00

Compare

ancientmarinerdev referenced this issue from a commit

2023-03-22 05:45:21 +01:00

Merge pull request 'Bundled lua with 5.7 crashes when you right click a farmer villager. math.random needs correct order' (#3576) from fix_villager_crash_with_bundle_lua into master