Open Bug 1006148 Opened 6 years ago Updated 6 years ago

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


(Core :: XPCOM, defect)

Other Branch
Not set




(Reporter: bjacob, Unassigned)


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().
You need to log in before you can comment on or make changes to this bug.