The future is now!
Created attachment 225854 [details] Patch
Comment on attachment 225854 [details] Patch r=me
Comment on attachment 225854 [details] Patch Clearing flags on attachment: 225854 Committed r165109: <http://trac.webkit.org/changeset/165109>
All reviewed patches have been landed. Closing bug.
Comment on attachment 225854 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=225854&action=review > Source/JavaScriptCore/heap/Heap.cpp:572 > + for (auto pair : m_protectedValues) This need to be auto& pair, otherwise it will copy the pair and work on the copy. Very bad!!! > Source/JavaScriptCore/heap/Heap.cpp:585 > + for (auto vector : m_tempSortingVectors) { > + for (auto valueStringPair : *vector) { Same thing. Needs to be auto&. > Source/JavaScriptCore/heap/Heap.h:454 > + for (auto pair : m_protectedValues) Same thing. Needs to be auto&.
Looks like Anders fixed it in bug 129745.