Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FaweCache Catching ArrayIndexOutOfBoundsException #2826

Open
2 tasks done
NonSwag opened this issue Jul 5, 2024 · 4 comments
Open
2 tasks done

FaweCache Catching ArrayIndexOutOfBoundsException #2826

NonSwag opened this issue Jul 5, 2024 · 4 comments
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"

Comments

@NonSwag
Copy link
Sponsor

NonSwag commented Jul 5, 2024

Server Implementation

Paper

Server Version

1.20.5/6

Describe the bug

When using a brush really fast FAWE throws an array index out of bounds exception
I can't tell whether this is a bug cased by goPaintAdvanced or within FAWE

To Reproduce

  1. Create a brush (using goPaintAdvanced)
  2. spam-click the brush

Expected behaviour

No exception

Screenshots / Videos

No response

Error log (if applicable)

https://pastebin.com/yWvtVrCW

Fawe Debugpaste

https://athion.net/ISPaster/paste/view/e56e09cbffb54ad1a7ca7f119a1864b8

Fawe Version

FastAsyncWorldEdit version 2.11.1-SNAPSHOT-843;5341b06

Checklist

Anything else?

The brush I used was from goPaintAdvanced and happens only when spamming edits

@NonSwag NonSwag added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Jul 5, 2024
@Folas1337
Copy link

I was also able to observe this with a default FAWE command like '//replace', here's the log:
https://mclo.gs/PSmkeJW

Paper 1.20.4 and FAWE 2.9.2 but this has been going on for a while already, it's not exactly new. I can find logs as old as 30 days that showed this exact error.

@dordsor21
Copy link
Member

I was also able to observe this with a default FAWE command like '//replace', here's the log: mclo.gs/PSmkeJW

Paper 1.20.4 and FAWE 2.9.2 but this has been going on for a while already, it's not exactly new. I can find logs as old as 30 days that showed this exact error.

Okay but that's a different issue and "it's been going on for a while" doesn't mean it's not been fixed? That's literally the opposite of how that works.

@Folas1337
Copy link

Oh I just thought it could've been relevant, nevermind me then. I just checked like the first line of the error, I should've been paying more attention.

@NonSwag
Copy link
Sponsor Author

NonSwag commented Jul 28, 2024

bump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"
3 participants