timeless Reporter
	Description • 19 years ago

found by coverity

i presume the second bit should null check, alternatively we should remove the earlier null check and replace it w/ an assertion

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 1 • 19 years ago

The assertion isn't necesary. Inspecting the code shows that it returns the address of mEntries[anIndex] (i.e., non-null) if anIndex < mCount. The call in question passes |mStack.mCount - 1|, so we can simply remove the null check.

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 3 • 19 years ago

Fixed.