]> git.lizzy.rs Git - minetest.git/blobdiff - src/craftdef.cpp
Report collisionMoveSimple for client and server. (#13105)
[minetest.git] / src / craftdef.cpp
index 6fc0ae934cc11e44a050da6245ebc0925f0cd131..eb703fdbee4251fad6ffc5b5a7663ae8adb7bf81 100644 (file)
@@ -22,14 +22,14 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "irrlichttypes.h"
 #include "log.h"
 #include <sstream>
-#include <set>
+#include <unordered_set>
 #include <algorithm>
 #include "gamedef.h"
 #include "inventory.h"
 #include "util/serialize.h"
 #include "util/string.h"
 #include "util/numeric.h"
-#include "strfnd.h"
+#include "util/strfnd.h"
 #include "exceptions.h"
 
 inline bool isGroupRecipeStr(const std::string &rec_name)
@@ -37,6 +37,15 @@ inline bool isGroupRecipeStr(const std::string &rec_name)
        return str_starts_with(rec_name, std::string("group:"));
 }
 
+static bool hasGroupItem(const std::vector<std::string> &recipe)
+{
+       for (const auto &item : recipe) {
+               if (isGroupRecipeStr(item))
+                       return true;
+       }
+       return false;
+}
+
 inline u64 getHashForString(const std::string &recipe_str)
 {
        /*errorstream << "Hashing craft string  \"" << recipe_str << '"';*/
@@ -49,17 +58,17 @@ static u64 getHashForGrid(CraftHashType type, const std::vector<std::string> &gr
                case CRAFT_HASH_TYPE_ITEM_NAMES: {
                        std::ostringstream os;
                        bool is_first = true;
-                       for (size_t i = 0; i < grid_names.size(); i++) {
-                               if (grid_names[i] != "") {
-                                       os << (is_first ? "" : "\n") << grid_names[i];
+                       for (const std::string &grid_name : grid_names) {
+                               if (!grid_name.empty()) {
+                                       os << (is_first ? "" : "\n") << grid_name;
                                        is_first = false;
                                }
                        }
                        return getHashForString(os.str());
                } case CRAFT_HASH_TYPE_COUNT: {
                        u64 cnt = 0;
-                       for (size_t i = 0; i < grid_names.size(); i++)
-                               if (grid_names[i] != "")
+                       for (const std::string &grid_name : grid_names)
+                               if (!grid_name.empty())
                                        cnt++;
                        return cnt;
                } case CRAFT_HASH_TYPE_UNHASHED:
@@ -67,6 +76,7 @@ static u64 getHashForGrid(CraftHashType type, const std::vector<std::string> &gr
        }
        // invalid CraftHashType
        assert(false);
+       return 0;
 }
 
 // Check if input matches recipe
@@ -89,7 +99,7 @@ static bool inputItemMatchesRecipe(const std::string &inp_name,
                                all_groups_match = false;
                                break;
                        }
-               } while (!f.atend());
+               } while (!f.at_end());
                if (all_groups_match)
                        return true;
        }
@@ -111,10 +121,9 @@ static std::vector<std::string> craftGetItemNames(
                const std::vector<std::string> &itemstrings, IGameDef *gamedef)
 {
        std::vector<std::string> result;
-       for (std::vector<std::string>::const_iterator
-                       it = itemstrings.begin();
-                       it != itemstrings.end(); it++) {
-               result.push_back(craftGetItemName(*it, gamedef));
+       result.reserve(itemstrings.size());
+       for (const auto &itemstring : itemstrings) {
+               result.push_back(craftGetItemName(itemstring, gamedef));
        }
        return result;
 }
@@ -124,10 +133,9 @@ static std::vector<std::string> craftGetItemNames(
                const std::vector<ItemStack> &items, IGameDef *gamedef)
 {
        std::vector<std::string> result;
-       for (std::vector<ItemStack>::const_iterator
-                       it = items.begin();
-                       it != items.end(); it++) {
-               result.push_back(it->name);
+       result.reserve(items.size());
+       for (const auto &item : items) {
+               result.push_back(item.name);
        }
        return result;
 }
@@ -137,11 +145,10 @@ static std::vector<ItemStack> craftGetItems(
                const std::vector<std::string> &items, IGameDef *gamedef)
 {
        std::vector<ItemStack> result;
-       for (std::vector<std::string>::const_iterator
-                       it = items.begin();
-                       it != items.end(); it++) {
-               result.push_back(ItemStack(std::string(*it), (u16)1,
-                       (u16)0, "", gamedef->getItemDefManager()));
+       result.reserve(items.size());
+       for (const auto &item : items) {
+               result.emplace_back(std::string(item), (u16)1,
+                       (u16)0, gamedef->getItemDefManager());
        }
        return result;
 }
@@ -155,11 +162,9 @@ static bool craftGetBounds(const std::vector<std::string> &items, unsigned int w
        bool success = false;
        unsigned int x = 0;
        unsigned int y = 0;
-       for (std::vector<std::string>::const_iterator
-                       it = items.begin();
-                       it != items.end(); it++) {
+       for (const std::string &item : items) {
                // Is this an actual item?
-               if (*it != "") {
+               if (!item.empty()) {
                        if (!success) {
                                // This is the first nonempty item
                                min_x = max_x = x;
@@ -186,11 +191,9 @@ static bool craftGetBounds(const std::vector<std::string> &items, unsigned int w
 // Removes 1 from each item stack
 static void craftDecrementInput(CraftInput &input, IGameDef *gamedef)
 {
-       for (std::vector<ItemStack>::iterator
-                       it = input.items.begin();
-                       it != input.items.end(); it++) {
-               if (it->count != 0)
-                       it->remove(1);
+       for (auto &item : input.items) {
+               if (item.count != 0)
+                       item.remove(1);
        }
 }
 
@@ -198,6 +201,7 @@ static void craftDecrementInput(CraftInput &input, IGameDef *gamedef)
 // Example: if replacements contains the pair ("bucket:bucket_water", "bucket:bucket_empty"),
 //   a water bucket will not be removed but replaced by an empty bucket.
 static void craftDecrementOrReplaceInput(CraftInput &input,
+               std::vector<ItemStack> &output_replacements,
                const CraftReplacements &replacements,
                IGameDef *gamedef)
 {
@@ -209,29 +213,31 @@ static void craftDecrementOrReplaceInput(CraftInput &input,
        // Make a copy of the replacements pair list
        std::vector<std::pair<std::string, std::string> > pairs = replacements.pairs;
 
-       for (std::vector<ItemStack>::iterator
-                       it = input.items.begin();
-                       it != input.items.end(); it++) {
-               if (it->count == 1) {
-                       // Find an appropriate replacement
-                       bool found_replacement = false;
-                       for (std::vector<std::pair<std::string, std::string> >::iterator
-                                       j = pairs.begin();
-                                       j != pairs.end(); j++) {
-                               if (it->name == craftGetItemName(j->first, gamedef)) {
-                                       it->deSerialize(j->second, gamedef->idef());
+       for (auto &item : input.items) {
+               // Find an appropriate replacement
+               bool found_replacement = false;
+               for (auto j = pairs.begin(); j != pairs.end(); ++j) {
+                       if (inputItemMatchesRecipe(item.name, j->first, gamedef->idef())) {
+                               if (item.count == 1) {
+                                       item.deSerialize(j->second, gamedef->idef());
                                        found_replacement = true;
                                        pairs.erase(j);
                                        break;
                                }
+
+                               ItemStack rep;
+                               rep.deSerialize(j->second, gamedef->idef());
+                               item.remove(1);
+                               found_replacement = true;
+                               pairs.erase(j);
+                               output_replacements.push_back(rep);
+                               break;
+
                        }
-                       // No replacement was found, simply decrement count to zero
-                       if (!found_replacement)
-                               it->remove(1);
-               } else if (it->count >= 2) {
-                       // Ignore replacements for items with count >= 2
-                       it->remove(1);
                }
+               // No replacement was found, simply decrement count by one
+               if (!found_replacement && item.count > 0)
+                       item.remove(1);
        }
 }
 
@@ -240,18 +246,17 @@ static std::string craftDumpMatrix(const std::vector<std::string> &items,
                unsigned int width)
 {
        std::ostringstream os(std::ios::binary);
-       os<<"{ ";
+       os << "{ ";
        unsigned int x = 0;
-       for(std::vector<std::string>::const_iterator
-                       it = items.begin();
-                       it != items.end(); it++, x++) {
+       for(std::vector<std::string>::size_type i = 0;
+                       i < items.size(); i++, x++) {
                if (x == width) {
-                       os<<"; ";
+                       os << "; ";
                        x = 0;
                } else if (x != 0) {
-                       os<<",";
+                       os << ",";
                }
-               os << '"' << (*it) << '"';
+               os << '"' << items[i] << '"';
        }
        os << " }";
        return os.str();
@@ -264,16 +269,15 @@ std::string craftDumpMatrix(const std::vector<ItemStack> &items,
        std::ostringstream os(std::ios::binary);
        os << "{ ";
        unsigned int x = 0;
-       for (std::vector<ItemStack>::const_iterator
-                       it = items.begin();
-                       it != items.end(); it++, x++) {
+       for (std::vector<ItemStack>::size_type i = 0;
+                       i < items.size(); i++, x++) {
                if (x == width) {
                        os << "; ";
                        x = 0;
                } else if (x != 0) {
-                       os<<",";
+                       os << ",";
                }
-               os << '"' << (it->getItemString()) << '"';
+               os << '"' << (items[i].getItemString()) << '"';
        }
        os << " }";
        return os.str();
@@ -284,6 +288,15 @@ std::string craftDumpMatrix(const std::vector<ItemStack> &items,
        CraftInput
 */
 
+bool CraftInput::empty() const
+{
+       for (const auto &item : items) {
+               if (!item.empty())
+                       return false;
+       }
+       return true;
+}
+
 std::string CraftInput::dump() const
 {
        std::ostringstream os(std::ios::binary);
@@ -312,10 +325,10 @@ std::string CraftReplacements::dump() const
        std::ostringstream os(std::ios::binary);
        os<<"{";
        const char *sep = "";
-       for (std::vector<std::pair<std::string, std::string> >::const_iterator
-                       it = pairs.begin();
-                       it != pairs.end(); it++) {
-               os << sep << '"' << (it->first) << "\"=>\"" << (it->second) << '"';
+       for (const auto &repl_p : pairs) {
+               os << sep
+                       << '"' << (repl_p.first)
+                       << "\"=>\"" << (repl_p.second) << '"';
                sep = ",";
        }
        os << "}";
@@ -326,6 +339,19 @@ std::string CraftReplacements::dump() const
        CraftDefinitionShaped
 */
 
+CraftDefinitionShaped::CraftDefinitionShaped(
+               const std::string &output_,
+               unsigned int width_,
+               const std::vector<std::string> &recipe_,
+               const CraftReplacements &replacements_):
+       output(output_), width(width_), recipe(recipe_), replacements(replacements_)
+{
+       if (hasGroupItem(recipe))
+               priority = PRIORITY_SHAPED_AND_GROUPS;
+       else
+               priority = PRIORITY_SHAPED;
+}
+
 std::string CraftDefinitionShaped::getName() const
 {
        return "shaped";
@@ -342,7 +368,7 @@ bool CraftDefinitionShaped::check(const CraftInput &input, IGameDef *gamedef) co
        if (inp_width == 0)
                return false;
        while (inp_names.size() % inp_width != 0)
-               inp_names.push_back("");
+               inp_names.emplace_back("");
 
        // Get input bounds
        unsigned int inp_min_x = 0, inp_max_x = 0, inp_min_y = 0, inp_max_y = 0;
@@ -361,7 +387,7 @@ bool CraftDefinitionShaped::check(const CraftInput &input, IGameDef *gamedef) co
        if (rec_width == 0)
                return false;
        while (rec_names.size() % rec_width != 0)
-               rec_names.push_back("");
+               rec_names.emplace_back("");
 
        // Get recipe bounds
        unsigned int rec_min_x=0, rec_max_x=0, rec_min_y=0, rec_max_y=0;
@@ -407,25 +433,10 @@ CraftInput CraftDefinitionShaped::getInput(const CraftOutput &output, IGameDef *
        return CraftInput(CRAFT_METHOD_NORMAL,width,craftGetItems(recipe,gamedef));
 }
 
-void CraftDefinitionShaped::decrementInput(CraftInput &input, IGameDef *gamedef) const
-{
-       craftDecrementOrReplaceInput(input, replacements, gamedef);
-}
-
-CraftHashType CraftDefinitionShaped::getHashType() const
+void CraftDefinitionShaped::decrementInput(CraftInput &input, std::vector<ItemStack> &output_replacements,
+        IGameDef *gamedef) const
 {
-       assert(hash_inited); // Pre-condition
-       bool has_group = false;
-       for (size_t i = 0; i < recipe_names.size(); i++) {
-               if (isGroupRecipeStr(recipe_names[i])) {
-                       has_group = true;
-                       break;
-               }
-       }
-       if (has_group)
-               return CRAFT_HASH_TYPE_COUNT;
-       else
-               return CRAFT_HASH_TYPE_ITEM_NAMES;
+       craftDecrementOrReplaceInput(input, output_replacements, replacements, gamedef);
 }
 
 u64 CraftDefinitionShaped::getHash(CraftHashType type) const
@@ -445,6 +456,11 @@ void CraftDefinitionShaped::initHash(IGameDef *gamedef)
                return;
        hash_inited = true;
        recipe_names = craftGetItemNames(recipe, gamedef);
+
+       if (hasGroupItem(recipe_names))
+               hash_type = CRAFT_HASH_TYPE_COUNT;
+       else
+               hash_type = CRAFT_HASH_TYPE_ITEM_NAMES;
 }
 
 std::string CraftDefinitionShaped::dump() const
@@ -460,6 +476,18 @@ std::string CraftDefinitionShaped::dump() const
        CraftDefinitionShapeless
 */
 
+CraftDefinitionShapeless::CraftDefinitionShapeless(
+               const std::string &output_,
+               const std::vector<std::string> &recipe_,
+               const CraftReplacements &replacements_):
+       output(output_), recipe(recipe_), replacements(replacements_)
+{
+       if (hasGroupItem(recipe))
+               priority = PRIORITY_SHAPELESS_AND_GROUPS;
+       else
+               priority = PRIORITY_SHAPELESS;
+}
+
 std::string CraftDefinitionShapeless::getName() const
 {
        return "shapeless";
@@ -472,11 +500,9 @@ bool CraftDefinitionShapeless::check(const CraftInput &input, IGameDef *gamedef)
 
        // Filter empty items out of input
        std::vector<std::string> input_filtered;
-       for (std::vector<ItemStack>::const_iterator
-                       it = input.items.begin();
-                       it != input.items.end(); it++) {
-               if (it->name != "")
-                       input_filtered.push_back(it->name);
+       for (const auto &item : input.items) {
+               if (!item.name.empty())
+                       input_filtered.push_back(item.name);
        }
 
        // If there is a wrong number of items in input, no match
@@ -528,25 +554,10 @@ CraftInput CraftDefinitionShapeless::getInput(const CraftOutput &output, IGameDe
        return CraftInput(CRAFT_METHOD_NORMAL, 0, craftGetItems(recipe, gamedef));
 }
 
-void CraftDefinitionShapeless::decrementInput(CraftInput &input, IGameDef *gamedef) const
-{
-       craftDecrementOrReplaceInput(input, replacements, gamedef);
-}
-
-CraftHashType CraftDefinitionShapeless::getHashType() const
+void CraftDefinitionShapeless::decrementInput(CraftInput &input, std::vector<ItemStack> &output_replacements,
+       IGameDef *gamedef) const
 {
-       assert(hash_inited); // Pre-condition
-       bool has_group = false;
-       for (size_t i = 0; i < recipe_names.size(); i++) {
-               if (isGroupRecipeStr(recipe_names[i])) {
-                       has_group = true;
-                       break;
-               }
-       }
-       if (has_group)
-               return CRAFT_HASH_TYPE_COUNT;
-       else
-               return CRAFT_HASH_TYPE_ITEM_NAMES;
+       craftDecrementOrReplaceInput(input, output_replacements, replacements, gamedef);
 }
 
 u64 CraftDefinitionShapeless::getHash(CraftHashType type) const
@@ -564,6 +575,11 @@ void CraftDefinitionShapeless::initHash(IGameDef *gamedef)
        hash_inited = true;
        recipe_names = craftGetItemNames(recipe, gamedef);
        std::sort(recipe_names.begin(), recipe_names.end());
+
+       if (hasGroupItem(recipe_names))
+               hash_type = CRAFT_HASH_TYPE_COUNT;
+       else
+               hash_type = CRAFT_HASH_TYPE_ITEM_NAMES;
 }
 
 std::string CraftDefinitionShapeless::dump() const
@@ -579,6 +595,12 @@ std::string CraftDefinitionShapeless::dump() const
        CraftDefinitionToolRepair
 */
 
+CraftDefinitionToolRepair::CraftDefinitionToolRepair(float additional_wear_):
+       additional_wear(additional_wear_)
+{
+       priority = PRIORITY_TOOLREPAIR;
+}
+
 static ItemStack craftToolRepair(
                const ItemStack &item1,
                const ItemStack &item2,
@@ -588,7 +610,7 @@ static ItemStack craftToolRepair(
        IItemDefManager *idef = gamedef->idef();
        if (item1.count != 1 || item2.count != 1 || item1.name != item2.name
                        || idef->get(item1.name).type != ITEM_TOOL
-                       || idef->get(item2.name).type != ITEM_TOOL) {
+                       || itemgroup_get(idef->get(item1.name).groups, "disable_repair") == 1) {
                // Failure
                return ItemStack();
        }
@@ -619,14 +641,12 @@ bool CraftDefinitionToolRepair::check(const CraftInput &input, IGameDef *gamedef
 
        ItemStack item1;
        ItemStack item2;
-       for (std::vector<ItemStack>::const_iterator
-                       it = input.items.begin();
-                       it != input.items.end(); it++) {
-               if (!it->empty()) {
+       for (const auto &item : input.items) {
+               if (!item.empty()) {
                        if (item1.empty())
-                               item1 = *it;
+                               item1 = item;
                        else if (item2.empty())
-                               item2 = *it;
+                               item2 = item;
                        else
                                return false;
                }
@@ -639,14 +659,12 @@ CraftOutput CraftDefinitionToolRepair::getOutput(const CraftInput &input, IGameD
 {
        ItemStack item1;
        ItemStack item2;
-       for (std::vector<ItemStack>::const_iterator
-                       it = input.items.begin();
-                       it != input.items.end(); it++) {
-               if (!it->empty()) {
+       for (const auto &item : input.items) {
+               if (!item.empty()) {
                        if (item1.empty())
-                               item1 = *it;
+                               item1 = item;
                        else if (item2.empty())
-                               item2 = *it;
+                               item2 = item;
                }
        }
        ItemStack repaired = craftToolRepair(item1, item2, additional_wear, gamedef);
@@ -656,11 +674,12 @@ CraftOutput CraftDefinitionToolRepair::getOutput(const CraftInput &input, IGameD
 CraftInput CraftDefinitionToolRepair::getInput(const CraftOutput &output, IGameDef *gamedef) const
 {
        std::vector<ItemStack> stack;
-       stack.push_back(ItemStack());
+       stack.emplace_back();
        return CraftInput(CRAFT_METHOD_COOKING, additional_wear, stack);
 }
 
-void CraftDefinitionToolRepair::decrementInput(CraftInput &input, IGameDef *gamedef) const
+void CraftDefinitionToolRepair::decrementInput(CraftInput &input, std::vector<ItemStack> &output_replacements,
+       IGameDef *gamedef) const
 {
        craftDecrementInput(input, gamedef);
 }
@@ -676,6 +695,19 @@ std::string CraftDefinitionToolRepair::dump() const
        CraftDefinitionCooking
 */
 
+CraftDefinitionCooking::CraftDefinitionCooking(
+               const std::string &output_,
+               const std::string &recipe_,
+               float cooktime_,
+               const CraftReplacements &replacements_):
+       output(output_), recipe(recipe_), cooktime(cooktime_), replacements(replacements_)
+{
+       if (isGroupRecipeStr(recipe))
+               priority = PRIORITY_SHAPELESS_AND_GROUPS;
+       else
+               priority = PRIORITY_SHAPELESS;
+}
+
 std::string CraftDefinitionCooking::getName() const
 {
        return "cooking";
@@ -688,11 +720,10 @@ bool CraftDefinitionCooking::check(const CraftInput &input, IGameDef *gamedef) c
 
        // Filter empty items out of input
        std::vector<std::string> input_filtered;
-       for (std::vector<ItemStack>::const_iterator
-                       it = input.items.begin();
-                       it != input.items.end(); it++) {
-               if (it->name != "")
-                       input_filtered.push_back(it->name);
+       for (const auto &item : input.items) {
+               const std::string &name = item.name;
+               if (!name.empty())
+                       input_filtered.push_back(name);
        }
 
        // If there is a wrong number of items in input, no match
@@ -704,7 +735,8 @@ bool CraftDefinitionCooking::check(const CraftInput &input, IGameDef *gamedef) c
        }
 
        // Check the single input item
-       return inputItemMatchesRecipe(input_filtered[0], recipe, gamedef->idef());
+       std::string rec_name = craftGetItemName(recipe, gamedef);
+       return inputItemMatchesRecipe(input_filtered[0], rec_name, gamedef->idef());
 }
 
 CraftOutput CraftDefinitionCooking::getOutput(const CraftInput &input, IGameDef *gamedef) const
@@ -719,30 +751,25 @@ CraftInput CraftDefinitionCooking::getInput(const CraftOutput &output, IGameDef
        return CraftInput(CRAFT_METHOD_COOKING,cooktime,craftGetItems(rec,gamedef));
 }
 
-void CraftDefinitionCooking::decrementInput(CraftInput &input, IGameDef *gamedef) const
+void CraftDefinitionCooking::decrementInput(CraftInput &input, std::vector<ItemStack> &output_replacements,
+       IGameDef *gamedef) const
 {
-       craftDecrementOrReplaceInput(input, replacements, gamedef);
-}
-
-CraftHashType CraftDefinitionCooking::getHashType() const
-{
-       if (isGroupRecipeStr(recipe_name))
-               return CRAFT_HASH_TYPE_COUNT;
-       else
-               return CRAFT_HASH_TYPE_ITEM_NAMES;
+       craftDecrementOrReplaceInput(input, output_replacements, replacements, gamedef);
 }
 
 u64 CraftDefinitionCooking::getHash(CraftHashType type) const
 {
        if (type == CRAFT_HASH_TYPE_ITEM_NAMES) {
                return getHashForString(recipe_name);
-       } else if (type == CRAFT_HASH_TYPE_COUNT) {
+       }
+
+       if (type == CRAFT_HASH_TYPE_COUNT) {
                return 1;
-       } else {
-               //illegal hash type for this CraftDefinition (pre-condition)
-               assert(false);
-               return 0;
        }
+
+       // illegal hash type for this CraftDefinition (pre-condition)
+       assert(false);
+       return 0;
 }
 
 void CraftDefinitionCooking::initHash(IGameDef *gamedef)
@@ -751,6 +778,11 @@ void CraftDefinitionCooking::initHash(IGameDef *gamedef)
                return;
        hash_inited = true;
        recipe_name = craftGetItemName(recipe, gamedef);
+
+       if (isGroupRecipeStr(recipe_name))
+               hash_type = CRAFT_HASH_TYPE_COUNT;
+       else
+               hash_type = CRAFT_HASH_TYPE_ITEM_NAMES;
 }
 
 std::string CraftDefinitionCooking::dump() const
@@ -767,6 +799,18 @@ std::string CraftDefinitionCooking::dump() const
        CraftDefinitionFuel
 */
 
+CraftDefinitionFuel::CraftDefinitionFuel(
+               const std::string &recipe_,
+               float burntime_,
+               const CraftReplacements &replacements_):
+       recipe(recipe_), burntime(burntime_), replacements(replacements_)
+{
+       if (isGroupRecipeStr(recipe_name))
+               priority = PRIORITY_SHAPELESS_AND_GROUPS;
+       else
+               priority = PRIORITY_SHAPELESS;
+}
+
 std::string CraftDefinitionFuel::getName() const
 {
        return "fuel";
@@ -779,11 +823,10 @@ bool CraftDefinitionFuel::check(const CraftInput &input, IGameDef *gamedef) cons
 
        // Filter empty items out of input
        std::vector<std::string> input_filtered;
-       for (std::vector<ItemStack>::const_iterator
-                       it = input.items.begin();
-                       it != input.items.end(); it++) {
-               if (it->name != "")
-                       input_filtered.push_back(it->name);
+       for (const auto &item : input.items) {
+               const std::string &name = item.name;
+               if (!name.empty())
+                       input_filtered.push_back(name);
        }
 
        // If there is a wrong number of items in input, no match
@@ -795,7 +838,8 @@ bool CraftDefinitionFuel::check(const CraftInput &input, IGameDef *gamedef) cons
        }
 
        // Check the single input item
-       return inputItemMatchesRecipe(input_filtered[0], recipe, gamedef->idef());
+       std::string rec_name = craftGetItemName(recipe, gamedef);
+       return inputItemMatchesRecipe(input_filtered[0], rec_name, gamedef->idef());
 }
 
 CraftOutput CraftDefinitionFuel::getOutput(const CraftInput &input, IGameDef *gamedef) const
@@ -810,30 +854,25 @@ CraftInput CraftDefinitionFuel::getInput(const CraftOutput &output, IGameDef *ga
        return CraftInput(CRAFT_METHOD_COOKING,(int)burntime,craftGetItems(rec,gamedef));
 }
 
-void CraftDefinitionFuel::decrementInput(CraftInput &input, IGameDef *gamedef) const
+void CraftDefinitionFuel::decrementInput(CraftInput &input, std::vector<ItemStack> &output_replacements,
+       IGameDef *gamedef) const
 {
-       craftDecrementOrReplaceInput(input, replacements, gamedef);
-}
-
-CraftHashType CraftDefinitionFuel::getHashType() const
-{
-       if (isGroupRecipeStr(recipe_name))
-               return CRAFT_HASH_TYPE_COUNT;
-       else
-               return CRAFT_HASH_TYPE_ITEM_NAMES;
+       craftDecrementOrReplaceInput(input, output_replacements, replacements, gamedef);
 }
 
 u64 CraftDefinitionFuel::getHash(CraftHashType type) const
 {
        if (type == CRAFT_HASH_TYPE_ITEM_NAMES) {
                return getHashForString(recipe_name);
-       } else if (type == CRAFT_HASH_TYPE_COUNT) {
+       }
+
+       if (type == CRAFT_HASH_TYPE_COUNT) {
                return 1;
-       } else {
-               //illegal hash type for this CraftDefinition (pre-condition)
-               assert(false);
-               return 0;
        }
+
+       // illegal hash type for this CraftDefinition (pre-condition)
+       assert(false);
+       return 0;
 }
 
 void CraftDefinitionFuel::initHash(IGameDef *gamedef)
@@ -842,7 +881,13 @@ void CraftDefinitionFuel::initHash(IGameDef *gamedef)
                return;
        hash_inited = true;
        recipe_name = craftGetItemName(recipe, gamedef);
+
+       if (isGroupRecipeStr(recipe_name))
+               hash_type = CRAFT_HASH_TYPE_COUNT;
+       else
+               hash_type = CRAFT_HASH_TYPE_ITEM_NAMES;
 }
+
 std::string CraftDefinitionFuel::dump() const
 {
        std::ostringstream os(std::ios::binary);
@@ -870,29 +915,21 @@ class CCraftDefManager: public IWritableCraftDefManager
        }
 
        virtual bool getCraftResult(CraftInput &input, CraftOutput &output,
-                       bool decrementInput, IGameDef *gamedef) const
+                       std::vector<ItemStack> &output_replacement, bool decrementInput,
+                       IGameDef *gamedef) const
        {
-               output.item = "";
-               output.time = 0;
-
-               // If all input items are empty, abort.
-               bool all_empty = true;
-               for (std::vector<ItemStack>::const_iterator
-                               it = input.items.begin();
-                               it != input.items.end(); it++) {
-                       if (!it->empty()) {
-                               all_empty = false;
-                               break;
-                       }
-               }
-               if (all_empty)
+               if (input.empty())
                        return false;
 
                std::vector<std::string> input_names;
                input_names = craftGetItemNames(input.items, gamedef);
                std::sort(input_names.begin(), input_names.end());
 
-               // Try hash types with increasing collision rate, and return if found.
+               // Try hash types with increasing collision rate
+               // while remembering the latest, highest priority recipe.
+               CraftDefinition::RecipePriority priority_best =
+                       CraftDefinition::PRIORITY_NO_RECIPE;
+               CraftDefinition *def_best = nullptr;
                for (int type = 0; type <= craft_hash_type_max; type++) {
                        u64 hash = getHashForGrid((CraftHashType) type, input_names);
 
@@ -900,8 +937,7 @@ class CCraftDefManager: public IWritableCraftDefManager
 
                        // We'd like to do "const [...] hash_collisions = m_craft_defs[type][hash];"
                        // but that doesn't compile for some reason. This does.
-                       std::map<u64, std::vector<CraftDefinition*> >::const_iterator
-                               col_iter = (m_craft_defs[type]).find(hash);
+                       auto col_iter = (m_craft_defs[type]).find(hash);
 
                        if (col_iter == (m_craft_defs[type]).end())
                                continue;
@@ -909,25 +945,37 @@ class CCraftDefManager: public IWritableCraftDefManager
                        const std::vector<CraftDefinition*> &hash_collisions = col_iter->second;
                        // Walk crafting definitions from back to front, so that later
                        // definitions can override earlier ones.
-                       for (std::vector<CraftDefinition*>::const_reverse_iterator
-                                       it = hash_collisions.rbegin();
-                                       it != hash_collisions.rend(); it++) {
-                               CraftDefinition *def = *it;
+                       for (std::vector<CraftDefinition*>::size_type
+                                       i = hash_collisions.size(); i > 0; i--) {
+                               CraftDefinition *def = hash_collisions[i - 1];
 
                                /*errorstream << "Checking " << input.dump() << std::endl
                                        << " against " << def->dump() << std::endl;*/
 
-                               if (def->check(input, gamedef)) {
-                                       // Get output, then decrement input (if requested)
-                                       output = def->getOutput(input, gamedef);
-                                       if (decrementInput)
-                                               def->decrementInput(input, gamedef);
-                                       /*errorstream << "Check RETURNS TRUE" << std::endl;*/
-                                       return true;
+                               CraftDefinition::RecipePriority priority = def->getPriority();
+                               if (priority > priority_best
+                                               && def->check(input, gamedef)) {
+                                       // Check if the crafted node/item exists
+                                       CraftOutput out = def->getOutput(input, gamedef);
+                                       ItemStack is;
+                                       is.deSerialize(out.item, gamedef->idef());
+                                       if (!is.isKnown(gamedef->idef())) {
+                                               infostream << "trying to craft non-existent "
+                                                       << out.item << ", ignoring recipe" << std::endl;
+                                               continue;
+                                       }
+
+                                       output = out;
+                                       priority_best = priority;
+                                       def_best = def;
                                }
                        }
                }
-               return false;
+               if (priority_best == CraftDefinition::PRIORITY_NO_RECIPE)
+                       return false;
+               if (decrementInput)
+                       def_best->decrementInput(input, output_replacement, gamedef);
+               return true;
        }
 
        virtual std::vector<CraftDefinition*> getCraftRecipes(CraftOutput &output,
@@ -935,8 +983,7 @@ class CCraftDefManager: public IWritableCraftDefManager
        {
                std::vector<CraftDefinition*> recipes;
 
-               std::map<std::string, std::vector<CraftDefinition*> >::const_iterator
-                       vec_iter = m_output_craft_definitions.find(output.item);
+               auto vec_iter = m_output_craft_definitions.find(output.item);
 
                if (vec_iter == m_output_craft_definitions.end())
                        return recipes;
@@ -945,26 +992,81 @@ class CCraftDefManager: public IWritableCraftDefManager
 
                recipes.reserve(limit ? MYMIN(limit, vec.size()) : vec.size());
 
-               for (std::vector<CraftDefinition*>::const_reverse_iterator
-                               it = vec.rbegin(); it != vec.rend(); ++it) {
+               for (std::vector<CraftDefinition*>::size_type i = vec.size();
+                               i > 0; i--) {
+                       CraftDefinition *def = vec[i - 1];
                        if (limit && recipes.size() >= limit)
                                break;
-                       recipes.push_back(*it);
+                       recipes.push_back(def);
                }
 
                return recipes;
        }
+
+       virtual bool clearCraftsByOutput(const CraftOutput &output, IGameDef *gamedef)
+       {
+               auto to_clear = m_output_craft_definitions.find(output.item);
+
+               if (to_clear == m_output_craft_definitions.end())
+                       return false;
+
+               for (auto def : to_clear->second) {
+                       // Recipes are not yet hashed at this point
+                       std::vector<CraftDefinition *> &defs = m_craft_defs[(int)CRAFT_HASH_TYPE_UNHASHED][0];
+                       defs.erase(std::remove(defs.begin(), defs.end(), def), defs.end());
+                       delete def;
+               }
+               m_output_craft_definitions.erase(to_clear);
+               return true;
+       }
+
+       virtual bool clearCraftsByInput(const CraftInput &input, IGameDef *gamedef)
+       {
+               if (input.empty())
+                       return false;
+
+               // Recipes are not yet hashed at this point
+               std::vector<CraftDefinition *> &defs = m_craft_defs[(int)CRAFT_HASH_TYPE_UNHASHED][0];
+               std::unordered_set<const CraftDefinition *> defs_to_remove;
+               std::vector<CraftDefinition *> new_defs;
+
+               for (auto def : defs) {
+                       if (def->check(input, gamedef))
+                               defs_to_remove.insert(def);
+                       else
+                               new_defs.push_back(def);
+               }
+
+               if (!defs_to_remove.empty()) {
+                       for (auto def : defs_to_remove)
+                               delete def;
+
+                       defs.swap(new_defs);
+
+                       for (auto &output : m_output_craft_definitions) {
+                               std::vector<CraftDefinition *> &outdefs = output.second;
+                               outdefs.erase(std::remove_if(outdefs.begin(), outdefs.end(), [&](const CraftDefinition* def) {
+                                       return defs_to_remove.find(def) != defs_to_remove.end();
+                               }), outdefs.end());
+                       }
+               }
+
+               return !defs_to_remove.empty();
+       }
+
        virtual std::string dump() const
        {
                std::ostringstream os(std::ios::binary);
                os << "Crafting definitions:\n";
-               for (int type = 0; type <= craft_hash_type_max; type++) {
-                       for (std::map<u64, std::vector<CraftDefinition*> >::const_iterator
-                                       it = (m_craft_defs[type]).begin();
-                                       it != (m_craft_defs[type]).end(); it++) {
-                               for (std::vector<CraftDefinition*>::const_iterator
-                                               iit = it->second.begin(); iit != it->second.end(); iit++) {
-                                       os << "type " << type << " hash " << it->first << (*iit)->dump() << "\n";
+               for (int type = 0; type <= craft_hash_type_max; ++type) {
+                       for (auto it = m_craft_defs[type].begin();
+                                       it != m_craft_defs[type].end(); ++it) {
+                               for (std::vector<CraftDefinition*>::size_type i = 0;
+                                               i < it->second.size(); i++) {
+                                       os << "type " << type
+                                               << " hash " << it->first
+                                               << " def " << it->second[i]->dump()
+                                               << "\n";
                                }
                        }
                }
@@ -972,8 +1074,8 @@ class CCraftDefManager: public IWritableCraftDefManager
        }
        virtual void registerCraft(CraftDefinition *def, IGameDef *gamedef)
        {
-               verbosestream << "registerCraft: registering craft definition: "
-                               << def->dump() << std::endl;
+               TRACESTREAM(<< "registerCraft: registering craft definition: "
+                               << def->dump() << std::endl);
                m_craft_defs[(int) CRAFT_HASH_TYPE_UNHASHED][0].push_back(def);
 
                CraftInput input;
@@ -983,15 +1085,12 @@ class CCraftDefManager: public IWritableCraftDefManager
        }
        virtual void clear()
        {
-               for (int type = 0; type <= craft_hash_type_max; type++) {
-                       for (std::map<u64, std::vector<CraftDefinition*> >::iterator
-                                       it = m_craft_defs[type].begin();
-                                       it != m_craft_defs[type].end(); it++) {
-                               for (std::vector<CraftDefinition*>::iterator
-                                               iit = it->second.begin(); iit != it->second.end(); iit++) {
-                                       delete *iit;
+               for (int type = 0; type <= craft_hash_type_max; ++type) {
+                       for (auto &it : m_craft_defs[type]) {
+                               for (auto &iit : it.second) {
+                                       delete iit;
                                }
-                               it->second.clear();
+                               it.second.clear();
                        }
                        m_craft_defs[type].clear();
                }
@@ -1000,11 +1099,9 @@ class CCraftDefManager: public IWritableCraftDefManager
        virtual void initHashes(IGameDef *gamedef)
        {
                // Move the CraftDefs from the unhashed layer into layers higher up.
-               for (std::vector<CraftDefinition*>::iterator
-                       it = (m_craft_defs[(int) CRAFT_HASH_TYPE_UNHASHED][0]).begin();
-                       it != (m_craft_defs[(int) CRAFT_HASH_TYPE_UNHASHED][0]).end(); it++) {
-                       CraftDefinition *def = *it;
-
+               std::vector<CraftDefinition *> &unhashed =
+                       m_craft_defs[(int) CRAFT_HASH_TYPE_UNHASHED][0];
+               for (auto def : unhashed) {
                        // Initialize and get the definition's hash
                        def->initHash(gamedef);
                        CraftHashType type = def->getHashType();
@@ -1013,16 +1110,16 @@ class CCraftDefManager: public IWritableCraftDefManager
                        // Enter the definition
                        m_craft_defs[type][hash].push_back(def);
                }
-               m_craft_defs[(int) CRAFT_HASH_TYPE_UNHASHED][0].clear();
+               unhashed.clear();
        }
 private:
-       //TODO: change both maps to unordered_map when c++11 can be used
-       std::vector<std::map<u64, std::vector<CraftDefinition*> > > m_craft_defs;
-       std::map<std::string, std::vector<CraftDefinition*> > m_output_craft_definitions;
+       std::vector<std::unordered_map<u64, std::vector<CraftDefinition*> > >
+               m_craft_defs;
+       std::unordered_map<std::string, std::vector<CraftDefinition*> >
+               m_output_craft_definitions;
 };
 
 IWritableCraftDefManager* createCraftDefManager()
 {
        return new CCraftDefManager();
 }
-