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

Prevent crash when eating stuff with internal furnace. #2267

Closed
wants to merge 2 commits into from

Conversation

Projects
None yet
3 participants
@1942rob
Copy link
Contributor

commented Jul 31, 2013

And display an eating message for it.

Edit. Also fixed bug where eating wielded stuff had no effect.

Issue #2266

@CataJenkins

This comment has been minimized.

Copy link

commented Jul 31, 2013

Build successful!
Refer to this link for build results: http://ci.narc.ro/job/Cataclysm-PullRequests/143/

Robert Sørlie
Prevent crash when eating stuff with int. furnace.
Also fixed bug where eating wielded stuff had no effect.
@CataJenkins

This comment has been minimized.

Copy link

commented Jul 31, 2013

Build successful!
Refer to this link for build results: http://ci.narc.ro/job/Cataclysm-PullRequests/144/

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jul 31, 2013

I'm going to pull the fix without the format change. We're avoiding whole-file reformats mostly, also planning on changing the coding style.
Actually if you look at the history of this particular bug, it was caused by a mass-reformat of the eat() method.

@1942rob

This comment has been minimized.

Copy link
Contributor Author

commented Aug 1, 2013

Ok. Had to AStyle it to see what I/the code was doing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.