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 the Error message when excessive Eat/Drink #31764

Merged
merged 2 commits into from
Jun 23, 2019
Merged

Fix the Error message when excessive Eat/Drink #31764

merged 2 commits into from
Jun 23, 2019

Conversation

kwl01skz
Copy link
Contributor

@kwl01skz kwl01skz commented Jun 23, 2019

Use the correct Boolean value to confirm the food.The old judgment will treat all "FOOD" as a drink.

Summary

SUMMARY: Bugfixes "Fix the Error message when excessive Eat/Drink"

Purpose of change

Now our protagonist has to drink a lot of bread and meatloaf.

Describe the solution

The old judgment can only judge the drink that can be eaten.
food.has_flag( "USE_EAT_VERB" )
but food can be eaten too.

Correct judgment
const bool edible = comest->comesttype == "FOOD" || food.has_flag( "USE_EAT_VERB" );

The old judgment will treat all "FOOD" as a drink.
@kwl01skz kwl01skz changed the title Use the correct Boolean value to confirm the food. Fix the Error message when excessive Eat/Drink Jun 23, 2019
@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Items: Food / Vitamins Comestibles and drinks labels Jun 23, 2019
@ZhilkinSerg ZhilkinSerg merged commit 6a5149d into CleverRaven:master Jun 23, 2019
@kwl01skz kwl01skz deleted the patch-19 branch July 27, 2019 05:03
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` Items: Food / Vitamins Comestibles and drinks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants