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

fix infinite attack loop when force attacking a non-hostile character #1166

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alanbirtles
Copy link
Contributor

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.

@qndel
Copy link
Member

qndel commented Mar 12, 2021

is this a vanilla bug?

@alanbirtles
Copy link
Contributor Author

could well be, its not one I remember encountering but it was 20 years ago!

@qndel
Copy link
Member

qndel commented Mar 12, 2021

can't you test in vanilla? :P

@alanbirtles
Copy link
Contributor Author

don't think it runs on my machine

@julealgon
Copy link
Contributor

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)?

@AJenbo
Copy link
Member

AJenbo commented Mar 13, 2021

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)

@galaxyhaxz
Copy link
Member

galaxyhaxz commented Mar 13, 2021

Tested in vanilla, I can't reproduce this bug.

Couldn't reproduce in DevilX either. Tried with Lachdanan

@alanbirtles
Copy link
Contributor Author

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.

@galaxyhaxz
Copy link
Member

Ok, so most likely it's specific to the barbarian and changes made with hellfire.

@alanbirtles
Copy link
Contributor Author

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

@julealgon
Copy link
Contributor

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.

@julealgon
Copy link
Contributor

@alanbirtles can you please provide a save file that can easily repro this issue?

@kphoenix137
Copy link
Collaborator

3 year old PR for an issue nobody except the author can reproduce. Close?
@AJenbo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants