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

Allow shooting out lights in the lab. #35724

Merged
merged 5 commits into from
Nov 28, 2019

Conversation

OzoneH3
Copy link
Member

@OzoneH3 OzoneH3 commented Nov 26, 2019

SUMMARY: Features "Allow shooting out lights in the lab."

Purpose of change

https://www.reddit.com/r/cataclysmdda/comments/dz1e8x/as_someone_who_spent_a_lot_of_time_in_labs/

Describe the solution

Added entry for t_thconc_floor_olight terrain to huge if else block.

Describe alternatives you've considered

Moving that whole if else mess into json. Was too lazy to do so.

Testing

Shot at light. Light dead.

src/map.cpp Outdated Show resolved Hide resolved
Co-Authored-By: Anton Burmistrov <Night_Pryanik@mail.ru>
@Night-Pryanik
Copy link
Contributor

Maybe add some chance to hit instead of guaranteed?

@ZhilkinSerg ZhilkinSerg added <Enhancement / Feature> New features, or enhancements on existing [C++] Changes (can be) made in C++. Previously named `Code` Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. labels Nov 26, 2019
@OzoneH3
Copy link
Member Author

OzoneH3 commented Nov 27, 2019

Gave it a 1 in 3 chance. At this point in the code there is no information about accuracy/distance/etc. available.

@ZhilkinSerg ZhilkinSerg merged commit 921e3b2 into CleverRaven:master Nov 28, 2019
@BreBat
Copy link

BreBat commented Dec 4, 2020

This is allowing "overhead lights" to block shots intended for targets on the ground

@ZhilkinSerg
Copy link
Contributor

This is allowing "overhead lights" to block shots intended for targets on the ground

No?

@OzoneH3
Copy link
Member Author

OzoneH3 commented Dec 4, 2020

As far as I remember this only gets called when shooting at an empty tile or the monster was missed with the shot.

@BreBat
Copy link

BreBat commented Dec 4, 2020

Ah, apologies if I was mistaken. I just noticed this behavior for the first time and assumed it had something to do with this pull.

image

@ZhilkinSerg
Copy link
Contributor

I guess you missed your target, so you hit a terrain instead. It could happen with other terrains too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` <Enhancement / Feature> New features, or enhancements on existing Fields / Furniture / Terrain / Traps Objects that are part of the map or its features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants