Fix cooking and fuel crafts with aliases

[dragonfireclient.git] / src / nodemetadata.cpp

diff --git a/src/nodemetadata.cpp b/src/nodemetadata.cpp
index 708a0809919f5474a21b2c79f65435af8ec620c5..b5052c3b8754122460330bd87f17b10661155973 100644 (file)
--- a/src/nodemetadata.cpp
+++ b/src/nodemetadata.cpp
@@ -49,8 +49,8 @@ void NodeMetadata::serialize(std::ostream &os, u8 version, bool disk) const
                if (!disk && priv)
                        continue;
 
-               os << serializeString(sv.first);
-               os << serializeLongString(sv.second);
+               os << serializeString16(sv.first);
+               os << serializeString32(sv.second);
                if (version >= 2)
                        writeU8(os, (priv) ? 1 : 0);
        }
@@ -63,8 +63,8 @@ void NodeMetadata::deSerialize(std::istream &is, u8 version)
        clear();
        int num_vars = readU32(is);
        for(int i=0; i<num_vars; i++){
-               std::string name = deSerializeString(is);
-               std::string var = deSerializeLongString(is);
+               std::string name = deSerializeString16(is);
+               std::string var = deSerializeString32(is);
                m_stringvars[name] = var;
                if (version >= 2) {
                        if (readU8(is) == 1)
@@ -112,13 +112,14 @@ int NodeMetadata::countNonPrivate() const
        NodeMetadataList
 */
 
-void NodeMetadataList::serialize(std::ostream &os, u8 blockver, bool disk) const
+void NodeMetadataList::serialize(std::ostream &os, u8 blockver, bool disk,
+       bool absolute_pos, bool include_empty) const
 {
        /*
                Version 0 is a placeholder for "nothing to see here; go away."
        */
 
-       u16 count = countNonEmpty();
+       u16 count = include_empty ? m_data.size() : countNonEmpty();
        if (count == 0) {
                writeU8(os, 0); // version
                return;
@@ -128,20 +129,29 @@ void NodeMetadataList::serialize(std::ostream &os, u8 blockver, bool disk) const
        writeU8(os, version);
        writeU16(os, count);
 
-       for (const auto &it : m_data) {
-               v3s16 p = it.first;
-               NodeMetadata *data = it.second;
-               if (data->empty())
+       for (NodeMetadataMap::const_iterator
+                       i = m_data.begin();
+                       i != m_data.end(); ++i) {
+               v3s16 p = i->first;
+               NodeMetadata *data = i->second;
+               if (!include_empty && data->empty())
                        continue;
 
-               u16 p16 = p.Z * MAP_BLOCKSIZE * MAP_BLOCKSIZE + p.Y * MAP_BLOCKSIZE + p.X;
-               writeU16(os, p16);
-
+               if (absolute_pos) {
+                       writeS16(os, p.X);
+                       writeS16(os, p.Y);
+                       writeS16(os, p.Z);
+               } else {
+                       // Serialize positions within a mapblock
+                       u16 p16 = (p.Z * MAP_BLOCKSIZE + p.Y) * MAP_BLOCKSIZE + p.X;
+                       writeU16(os, p16);
+               }
                data->serialize(os, version, disk);
        }
 }
 
-void NodeMetadataList::deSerialize(std::istream &is, IItemDefManager *item_def_mgr)
+void NodeMetadataList::deSerialize(std::istream &is,
+       IItemDefManager *item_def_mgr, bool absolute_pos)
 {
        clear();
 
@@ -162,15 +172,19 @@ void NodeMetadataList::deSerialize(std::istream &is, IItemDefManager *item_def_m
        u16 count = readU16(is);
 
        for (u16 i = 0; i < count; i++) {
-               u16 p16 = readU16(is);
-
                v3s16 p;
-               p.Z = p16 / MAP_BLOCKSIZE / MAP_BLOCKSIZE;
-               p16 &= MAP_BLOCKSIZE * MAP_BLOCKSIZE - 1;
-               p.Y = p16 / MAP_BLOCKSIZE;
-               p16 &= MAP_BLOCKSIZE - 1;
-               p.X = p16;
-
+               if (absolute_pos) {
+                       p.X = readS16(is);
+                       p.Y = readS16(is);
+                       p.Z = readS16(is);
+               } else {
+                       u16 p16 = readU16(is);
+                       p.X = p16 & (MAP_BLOCKSIZE - 1);
+                       p16 /= MAP_BLOCKSIZE;
+                       p.Y = p16 & (MAP_BLOCKSIZE - 1);
+                       p16 /= MAP_BLOCKSIZE;
+                       p.Z = p16;
+               }
                if (m_data.find(p) != m_data.end()) {
                        warningstream << "NodeMetadataList::deSerialize(): "
                                        << "already set data at position " << PP(p)
@@ -192,19 +206,18 @@ NodeMetadataList::~NodeMetadataList()
 std::vector<v3s16> NodeMetadataList::getAllKeys()
 {
        std::vector<v3s16> keys;
-
-       std::map<v3s16, NodeMetadata *>::const_iterator it;
-       for (it = m_data.begin(); it != m_data.end(); ++it)
-               keys.push_back(it->first);
+       keys.reserve(m_data.size());
+       for (const auto &it : m_data)
+               keys.push_back(it.first);
 
        return keys;
 }
 
 NodeMetadata *NodeMetadataList::get(v3s16 p)
 {
-       std::map<v3s16, NodeMetadata *>::const_iterator n = m_data.find(p);
+       NodeMetadataMap::const_iterator n = m_data.find(p);
        if (n == m_data.end())
-               return NULL;
+               return nullptr;
        return n->second;
 }
 
@@ -212,7 +225,8 @@ void NodeMetadataList::remove(v3s16 p)
 {
        NodeMetadata *olddata = get(p);
        if (olddata) {
-               delete olddata;
+               if (m_is_metadata_owner)
+                       delete olddata;
                m_data.erase(p);
        }
 }
@@ -220,14 +234,15 @@ void NodeMetadataList::remove(v3s16 p)
 void NodeMetadataList::set(v3s16 p, NodeMetadata *d)
 {
        remove(p);
-       m_data.insert(std::make_pair(p, d));
+       m_data.emplace(p, d);
 }
 
 void NodeMetadataList::clear()
 {
-       std::map<v3s16, NodeMetadata*>::iterator it;
-       for (it = m_data.begin(); it != m_data.end(); ++it) {
-               delete it->second;
+       if (m_is_metadata_owner) {
+               NodeMetadataMap::const_iterator it;
+               for (it = m_data.begin(); it != m_data.end(); ++it)
+                       delete it->second;
        }
        m_data.clear();
 }
@@ -235,9 +250,8 @@ void NodeMetadataList::clear()
 int NodeMetadataList::countNonEmpty() const
 {
        int n = 0;
-       std::map<v3s16, NodeMetadata*>::const_iterator it;
-       for (it = m_data.begin(); it != m_data.end(); ++it) {
-               if (!it->second->empty())
+       for (const auto &it : m_data) {
+               if (!it.second->empty())
                        n++;
        }
        return n;