diff options
| author | frsyuki <frsyuki@users.sourceforge.jp> | 2009-06-03 22:01:27 +0900 |
|---|---|---|
| committer | frsyuki <frsyuki@users.sourceforge.jp> | 2009-06-03 22:01:27 +0900 |
| commit | 861721314965cf89beaa17f3dfcdce90ccd8d089 (patch) | |
| tree | 2c3d6d54ca271c2dd925f2dee765fcc426610b86 | |
| parent | b0062a7f6f47b9a4340d14c6fb0a3ae0f60c8259 (diff) | |
| download | msgpack-python-861721314965cf89beaa17f3dfcdce90ccd8d089.tar.gz | |
cpp: fix map<K, V> converter
| -rw-r--r-- | cpp/type/map.hpp | 10 | ||||
| -rw-r--r-- | ruby/test_case.rb | 1 |
2 files changed, 6 insertions, 5 deletions
diff --git a/cpp/type/map.hpp b/cpp/type/map.hpp index c136d53..552de57 100644 --- a/cpp/type/map.hpp +++ b/cpp/type/map.hpp @@ -80,13 +80,13 @@ inline std::map<K, V> operator>> (object o, std::map<K, V>& v) for(; p != pend; ++p) { K key; p->key.convert(&key); - typename std::map<K,V>::iterator it(v.find(key)); - if(it != v.end()) { + typename std::map<K,V>::iterator it(v.lower_bound(key)); + if(it != v.end() && !(key < it->first)) { + p->val.convert(&it->second); + } else { V val; p->val.convert(&val); - it->insert( std::pair<K,V>(key, val) ); - } else { - p->val.convert(&it->second); + v.insert(it, std::pair<K,V>(key, val)); } } return v; diff --git a/ruby/test_case.rb b/ruby/test_case.rb index 2d897df..4fbcea3 100644 --- a/ruby/test_case.rb +++ b/ruby/test_case.rb @@ -219,6 +219,7 @@ class MessagePackTestFormat < Test::Unit::TestCase def match(obj, buf) assert_equal(obj.to_msgpack, buf) + assert_equal(MessagePack::unpack(buf), obj) end end |
