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

Digitigrade Mutation Line-of-Sight Fix #66764

Merged
merged 9 commits into from
Jul 9, 2023

Conversation

GalacticApple
Copy link
Contributor

@GalacticApple GalacticApple commented Jul 8, 2023

Summary

Bugfixes "Updates the mutation so that players can't see over objects while running on all-fours"

Purpose of change

The mutation has it so that you run on all-fours while running. However, you could still see over objects like you were standing upright.

Describe the solution

I made some changes to the character.cpp/h files and the lightmap.cpp file. Specifically, I added in a is_runallfours function which is then used in the lightmap.cpp code for limiting sight while crouching.

Describe alternatives you've considered

Testing

It works on my personal build of CDDA.

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies [C++] Changes (can be) made in C++. Previously named `Code` labels Jul 8, 2023
@GalacticApple GalacticApple changed the title Digitigrade Mutation Update Digitigrade Mutation Line-of-Sight Fix Jul 8, 2023
@github-actions github-actions bot added the <Bugfix> This is a fix for a bug (or closes open issue) label Jul 8, 2023
src/character.cpp Outdated Show resolved Hide resolved
src/character.cpp Outdated Show resolved Hide resolved
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Jul 8, 2023
GalacticApple and others added 2 commits July 8, 2023 18:41
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions and removed astyled astyled PR, label is assigned by github actions labels Jul 8, 2023
@alp7292
Copy link
Contributor

alp7292 commented Jul 9, 2023

does it also hides you when in cover like crouching

@Maleclypse Maleclypse merged commit a566717 into CleverRaven:master Jul 9, 2023
@GalacticApple
Copy link
Contributor Author

does it also hides you when in cover like crouching

yep

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants