Open Bug 1006148 Opened 11 years ago Updated 2 years ago

Go over users of nsTArray::NoIndex and consider instead making use of RemoveElement, Contains, etc, instead of using NoIndex

Categories

(Core :: XPCOM, defect)

Product:
Core
Component:
XPCOM
Version:
Other Branch
Type:
defect

Tracking

()

People

(Reporter: bjacob, Unassigned)

Details

Having the largest value in the index type magically represent "no index" is not perfect; it's been a source of bugs while working on bug 1004098 where we changed the index type. In bug 1004098 comment 11 Nathan suggested that some of the existing users of nsTArray::NoIndex might actually be better off not using NoIndex and instead making better use of nsTArray functions such as RemoveElement() or Contains().
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.