From 0195f436416c1abf6d287a3a4360ccc8d029d7f3 Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Wed, 8 Aug 2012 11:52:09 -0700 Subject: [PATCH] Bug 781289 - Remove unecessary check that let variable name matches are atoms. r=luke --- js/src/jsopcode.cpp | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/js/src/jsopcode.cpp b/js/src/jsopcode.cpp index 921c47648fe..4e418ea7706 100644 --- a/js/src/jsopcode.cpp +++ b/js/src/jsopcode.cpp @@ -6093,13 +6093,8 @@ ExpressionDecompiler::findLetVar(jsbytecode *pc, unsigned depth) if (uint32_t(depth - blockDepth) < uint32_t(blockCount)) { for (Shape::Range r(block.lastProperty()); !r.empty(); r.popFront()) { const Shape &shape = r.front(); - if (shape.shortid() == int(depth - blockDepth)) { - // !JSID_IS_ATOM(shape.propid()) happens for empty - // destructuring variables in lets. They can be safely - // ignored. - if (JSID_IS_ATOM(shape.propid())) - return JSID_TO_ATOM(shape.propid()); - } + if (shape.shortid() == int(depth - blockDepth)) + return JSID_TO_ATOM(shape.propid()); } } chain = chain->getParent();