projects / folly.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 29ca682) | patch
Fix dynamic::insert
authorPavlo Kushnir <pavlo@fb.com>
Thu, 27 Nov 2014 03:28:18 +0000 (19:28 -0800)
committerDave Watson <davejwatson@fb.com>
Thu, 11 Dec 2014 15:59:05 +0000 (07:59 -0800)
commit78b2965a1cf7eedf1107c1bb54f010f1215d9cb4
tree07f5cbfad837666458128b951a3bc981765297b0tree | snapshot
parent29ca6827b5387ebeeffcd1407492685a3327df25commit | diff
Fix dynamic::insert

Summary: folly::dynamic::insert is broken. When we try to insert an rvalue and the key already exists in dynamic, insert will replace the value with empty dynamic.

Test Plan: unit test attached

Reviewed By: stepan@fb.com

Subscribers: trunkagent, njormrod, folly-diffs@

FB internal diff: D1704995

Signature: t1:1704995:1417053631:8e0163693df721c54b016f8c27e81e7bf60194cb
folly/dynamic-inl.h diff | blob | history
folly/test/DynamicTest.cpp diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.