Refactor parser rollback logic
Created attachment 215229 [details] Patch
Committed r158074: <http://trac.webkit.org/changeset/158074>
Comment on attachment 215229 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=215229&action=review > Source/JavaScriptCore/parser/Parser.cpp:202 > + auto savePoint = createSavePoint(); In cases like this, I think the code is more self-documenting, and not harder to write, if it says "SavePoint savePoint = ...".
Comment on attachment 215229 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=215229&action=review R=me too fwiw. >> Source/JavaScriptCore/parser/Parser.cpp:202 >> + auto savePoint = createSavePoint(); > > In cases like this, I think the code is more self-documenting, and not harder to write, if it says "SavePoint savePoint = ...". +1 This is one of those situations where types are cool.