556559 - Fix warnings in TestTArray.cpp - "warning: comparison between signed and unsigned integer expressions"

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Daniel Holbert [:dholbert]]

{
> TestTArray.cpp: In function 'PRBool TestTArray::test_string_array()':
> TestTArray.cpp:318: warning: comparison between signed and unsigned integer expressions
> TestTArray.cpp: In function 'PRBool TestTArray::test_indexof()':
> TestTArray.cpp:521: warning: comparison between signed and unsigned integer expressions
> TestTArray.cpp: In function 'PRBool TestTArray::test_basic_array(ElementType*, PRUint32, const ElementType&) [with ElementType = int]':
> TestTArray.cpp:175:   instantiated from here
> TestTArray.cpp:105: warning: comparison between signed and unsigned integer expressions
> TestTArray.cpp: In function 'PRBool TestTArray::test_basic_array(ElementType*, PRUint32, const ElementType&) [with ElementType = PRInt64]':
> TestTArray.cpp:180:   instantiated from here
> TestTArray.cpp:105: warning: comparison between signed and unsigned integer expressions
> TestTArray.cpp: In function 'PRBool TestTArray::test_basic_array(ElementType*, PRUint32, const ElementType&) [with ElementType = char]':
> TestTArray.cpp:185:   instantiated from here
> TestTArray.cpp:105: warning: comparison between signed and unsigned integer expressions
> TestTArray.cpp: In function 'PRBool TestTArray::test_basic_array(ElementType*, PRUint32, const ElementType&) [with ElementType = PRUint32]':
> TestTArray.cpp:190:   instantiated from here
> TestTArray.cpp:105: warning: comparison between signed and unsigned integer expressions
> TestTArray.cpp: In function 'PRBool TestTArray::test_basic_array(ElementType*, PRUint32, const ElementType&) [with ElementType = nsCString]':
> TestTArray.cpp:325:   instantiated from here
> TestTArray.cpp:105: warning: comparison between signed and unsigned integer expressions
> TestTArray.cpp: In function 'PRBool TestTArray::test_basic_array(ElementType*, PRUint32, const ElementType&) [with ElementType = nsCOMPtr<nsIFile>]':
> TestTArray.cpp:366:   instantiated from here
> TestTArray.cpp:105: warning: comparison between signed and unsigned integer expressions
}

This is due to lines like:
{
318   if (strArray.BinaryIndexOf(EmptyCString()) != -1)
319     return PR_FALSE;
}

Note that BinaryIndexOf returns an unsigned value.  We should be comparing it against nsTArray::NoIndex instead of -1.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: fix

Comment 2

[Daniel Holbert [:dholbert]]

http://hg.mozilla.org/mozilla-central/rev/ccce7bebe7d9