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

Gasoline lanterns #1117

Merged
merged 5 commits into from
May 22, 2013
Merged

Conversation

alex-gh
Copy link
Contributor

@alex-gh alex-gh commented May 21, 2013

This implements #76

I couldn't bring myself to call them hurricane lanterns though: those are normally kerosene powered and are quite dangerous when filled with gasoline.

@HeatherSoron
Copy link
Contributor

Looks like there's no legitimate in-game way to acquire these. They should be added to the spawn lists (mapitemsdef.cpp), or at least made craftable.

@alex-gh
Copy link
Contributor Author

alex-gh commented May 21, 2013

I knew I was missing something. I have added them to spawns that seemed feasible. Thanks!

I'll probably create a recipe for them in the near future ( Survival + Mechanics? )

@HeatherSoron
Copy link
Contributor

Alright, thanks :).

And... ah, right, should've spotted this sooner: they don't actually work. The LIGHT_N flags are all special-cased, and LIGHT_3 doesn't exist yet, so it's not handled by the lightmap. On top of that, player::active_light doesn't even use those flags; it's special-cased based on the item IDs.

This is really an awkward system.

@alex-gh
Copy link
Contributor Author

alex-gh commented May 21, 2013

Ah serves me right for coding by template ;)

I Intended to make it a LIGHT_4 like a candle, but put in a 3 by accident.

I'll figure it out in the next days and fix it.

@HeatherSoron
Copy link
Contributor

Cool, thanks :).

@HeatherSoron
Copy link
Contributor

Man these are weak - worse than what Night Vision gives you. But, the lighting system in general is wonky, so I'd be fine with addressing that as a separate issue.

HeatherSoron added a commit that referenced this pull request May 22, 2013
@HeatherSoron HeatherSoron merged commit 723002d into CleverRaven:master May 22, 2013
@alex-gh
Copy link
Contributor Author

alex-gh commented May 22, 2013

Well yes, I thought of it more as a tool to read/sew in the dark, not as something you would take to go adventuring.

@kevingranade
Copy link
Member

Yea, I've used hurricane lanterns, you can barely read by them, and they
only dimply light up a medium-sized room in a house
On May 22, 2013 12:13 AM, "alex-gh" notifications@github.com wrote:

Well yes, I thought of it more as a tool to read/sew in the dark, not as
something you would take to go adventuring.


Reply to this email directly or view it on GitHubhttps://github.com/TheDarklingWolf/Cataclysm-DDA/pull/1117#issuecomment-18257889
.

@HeatherSoron
Copy link
Contributor

Ah, cool, so they're supposed to be like that.

@alex-gh alex-gh deleted the hurricane_lanterns branch May 22, 2013 05:57
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.

3 participants