-
Notifications
You must be signed in to change notification settings - Fork 793
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
fix infinite attack loop when force attacking a non-hostile character #1166
base: master
Are you sure you want to change the base?
Conversation
is this a vanilla bug? |
could well be, its not one I remember encountering but it was 20 years ago! |
can't you test in vanilla? :P |
don't think it runs on my machine |
By "force attack" do you mean shift click in the opposite direction? Does this happen with every class/weapon? Does it happen consistently regardless of save (did you check with different characters)? |
looks like a reasonable fix, i actually found something similar the other day. If attack a barral next to Lachdanan and then click him before the animation is over you will perform attacks on him (he wong get hit) |
Tested in vanilla, I can't reproduce this bug. Couldn't reproduce in DevilX either. Tried with Lachdanan |
This was with bard dual welding swords, I didn't try other characters. Yes I do mean shift clicking, note that shift clicking on the character directly doesn't trigger an attack, you have to click behind the character so that you are attacking in their direction but not attempting to attack them directly. Looking at the code I imagine this will happen with any melee weapon. |
Ok, so most likely it's specific to the barbarian and changes made with hellfire. |
yes, it does seem to only affect the bard, I couldn't reproduce with a warrior. With a bard it happens with single or dual wielded melee weapons |
Fix looks a bit arbitrary/hacky to me. I wonder if there is a cleaner way to go about this. If the problem only occurs with bard, it doesn't make sense to me to do this this way. Why does it happen only with the bard in the first place? I don't feel comfortable in making fixes while we don't clearly understand the problem at hand. |
@alanbirtles can you please provide a save file that can easily repro this issue? |
3 year old PR for an issue nobody except the author can reproduce. Close? |
Not sure if this is the right fix but fixes the problem for me. Reproduce by standing next to a non hostile character (I found the bug with Lachdanan), putting the cursor the opposite side of them then force attack. An infinite loop of attacks starts and there doesn't seem to be any way to break out of it.