ArrayList.contains uses deprecated equality test

[guest]

1.0b definition of ArrayList?.contains uses the == operator which has been deprecated as of JS 1.5 in favor of JSObject.equals.

this.contains=function(/* object */obj){
        //	summary
	//	Check to see if the passed object is a member in the ArrayList
	for(var i=0; i < items.length; i++){
		if(items[i] == obj) {
			return true;	//	bool
		}
	}
	return false;	//	bool
};

[guest]

Looking at Mozilla reference. Invalid.

[Tom Trenka]

Um...ok. So you're saying (no idea who, no email attached) using a standard == operator to compare object references has been deprecated and is wrong? I hate to say this but this is a standard operator, and I think the bug is kind of an unnecessary nitpick--particularly when you take into account JScript.

If anything, it should be the === operator.

The object you are referring to--JSObject--is actually a Java representation of a JavaScript? object. Why we would tap into a Java engine to test something standard like this is beyond me.

Marking this bug as invalid. Appreciate the look-over but not seeing anything here (particularly in terms of an object method) that would warrant changing this.