Scarpet-edit icon indicating copy to clipboard operation
Scarpet-edit copied to clipboard

I find some bugs

Open ch-yx opened this issue 3 years ago • 1 comments

https://github.com/gnembon/fabric-carpet/issues/1118#issuecomment-927389553

@altrisi thanks you. but after my reading and testing that script, i find some issues.

https://github.com/Ghoulboy78/Scarpet-edit/blob/64543f220242657c0086c20a4ccfb138bc7dcce5/se.sc#L1943-L1945

seems that the area is not correct. maybe pos2=pos2+1 could fix that ?

https://github.com/Ghoulboy78/Scarpet-edit/blob/64543f220242657c0086c20a4ccfb138bc7dcce5/se.sc#L1957-L1961

seems that the one who writes the code incorrectly assumes that the map will remain its order.

https://github.com/Ghoulboy78/Scarpet-edit/blob/64543f220242657c0086c20a4ccfb138bc7dcce5/se.sc#L1963

I am concerned that the unnecessary use of parse_nbt here will disrupt some of the nbt types.

ch-yx avatar Sep 28 '21 15:09 ch-yx

Ye... that part of the code is overdue for a rewrite anyways... We just havent had the time. I have fixed some of the issues on a private branch that I haven't touched for a while.

Ghoulboy78 avatar Sep 29 '21 12:09 Ghoulboy78