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 issue with auto_aim #40662

Merged
merged 3 commits into from
May 22, 2020
Merged

Conversation

Ramza13
Copy link
Contributor

@Ramza13 Ramza13 commented May 18, 2020

Summary

SUMMARY: Bugfixes "Fix problems with auto aim"

Purpose of change

Fixes #38285
Fixes auto aim for ranged weapons not targeting through glass, chain link fences, etc while preventing melee attacks trying to attack through things they can't go through.

Describe the solution

Added param to differentiate between ranged and melee auto attacks, reverted ranged behavior and modified reach attacks to only target things they could hit

Describe alternatives you've considered

Testing

Auto targeted through fence with gun, couldn't do so through glass with melee.

Additional context

src/ranged.cpp Outdated Show resolved Hide resolved
@curstwist curstwist added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Mechanics: Aiming Aiming, especially aiming balance labels May 18, 2020
src/character.cpp Outdated Show resolved Hide resolved
@ZhilkinSerg ZhilkinSerg merged commit ac61da2 into CleverRaven:master May 22, 2020
@Ramza13 Ramza13 deleted the auto_aim_correction branch May 23, 2020 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Mechanics: Aiming Aiming, especially aiming balance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto-target does not auto-target through display racks
4 participants