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 several combat bugs #204

Merged
merged 2 commits into from
Dec 6, 2022
Merged

fix several combat bugs #204

merged 2 commits into from
Dec 6, 2022

Conversation

lexurco
Copy link
Contributor

@lexurco lexurco commented Dec 5, 2022

Hi,

I tried sending my diff to the owner, but I don't seem to be getting
an answer. Haven't exchanged diffs over github previously, hope it's
ok. The list should be mostly self-explanatory:

  • fixed crit misses becoming 0-damage crit hits;
  • fixed wrong messages after " and ";
  • fixed the silent death perk benefitting everyone but the player;

Alexander Arkhipov and others added 2 commits December 5, 2022 20:29
- fixed crit misses becoming 0-damage crit hits;
- fixed wrong messages after " and ";
- fixed the silent death perk benefitting everyone but the player;
@alexbatalov
Copy link
Owner

Hey, thanks for your submission. Your assumption on Silent Death is wrong. Initially whoHitMe is null, with your fix Silent Death will never be applied unless dude already hit target with other attack, thus defeating the purpose of this perk entirely. The check guards against continuous use of Silent Death on the same target.

All other fixes are bugs, verified in the original binary.

@alexbatalov alexbatalov merged commit d49869f into alexbatalov:main Dec 6, 2022
@lexurco lexurco deleted the fix-combat branch December 6, 2022 17:05
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.

2 participants