AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Remove stray comment from ast.cc

Browse Source 
This comment used to have a (commented-out) ASSERT attached to it.
But that ASSERT was removed years ago, so the comment is now a
non-sequitar.

Review URL: https://codereview.chromium.org/918453002

Cr-Commit-Position: refs/heads/master@{#26587}
This commit is contained in:
adamk 2015-02-11 07:36:27 -08:00 committed by Commit bot
parent b5e235661f
commit 4043c45f46
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/ast.cc
View File

@ -84,11 +84,6 @@ VariableProxy::VariableProxy(Zone* zone, const AstRawString* name, bool is_this,
void VariableProxy::BindTo(Variable* var) { void VariableProxy::BindTo(Variable* var) {
DCHECK(!FLAG_harmony_modules || interface_->IsUnified(var->interface())); DCHECK(!FLAG_harmony_modules || interface_->IsUnified(var->interface()));
DCHECK((is_this() && var->is_this()) || raw_name() == var->raw_name()); DCHECK((is_this() && var->is_this()) || raw_name() == var->raw_name());
// Ideally CONST-ness should match. However, this is very hard to achieve
// because we don't know the exact semantics of conflicting (const and
// non-const) multiple variable declarations, const vars introduced via
// eval() etc. Const-ness and variable declarations are a complete mess
// in JS. Sigh...
set_var(var); set_var(var);
set_is_resolved(); set_is_resolved();
var->set_is_used(); var->set_is_used();