Consider adding static analysis to prohibit Rooted locals inside loops

NEW
Unassigned

Status

()

Core
JavaScript Engine
P3
normal
a year ago
7 months ago

People

(Reporter: anba, Unassigned)

Tracking

({triage-deferred})

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

a year ago
In bug 1321789, comment #4, Waldo wrote:
> Could you file a bug (CC me on it) to add static analysis to consider
> prohibiting Rooted locals inside loops?  I'd be interested in seeing the
> cases where we fall down, because my guess is this wouldn't really hinder
> readability that much (and if it did, we could have the Rooted outside the
> loop and a renamed Rooted& inside it), and it's something we're looking out
> for anyway, so we might as well get proper enforcement of it.
Keywords: triage-deferred
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.