In gcc/objc/: 2011-10-18 Mikael Pettersson <mikpe@it.uu.se>

In gcc/objc/:
2011-10-18  Mikael Pettersson  <mikpe@it.uu.se>

	PR objc/50743
	* objc-act.c (check_duplicates): Cast TREE_VEC_LENGTH result to
	size_t to avoid signed/unsigned
	comparison.
	(insert_method_into_method_map): Likewise.

From-SVN: r180132
This commit is contained in:
Mikael Pettersson 2011-10-18 13:31:45 +02:00 committed by Nicola Pero
parent 6cc4833a49
commit 5b6b2942cd
2 changed files with 12 additions and 4 deletions

View File

@ -1,3 +1,11 @@
2011-10-18 Mikael Pettersson <mikpe@it.uu.se>
PR objc/50743
* objc-act.c (check_duplicates): Cast TREE_VEC_LENGTH result to
size_t to avoid signed/unsigned
comparison.
(insert_method_into_method_map): Likewise.
2011-10-14 Nicola Pero <nicola.pero@meta-innovation.com> 2011-10-14 Nicola Pero <nicola.pero@meta-innovation.com>
* objc-map.h: New file. * objc-map.h: New file.

View File

@ -5070,7 +5070,7 @@ check_duplicates (tree method, int methods, int is_class)
alignment. */ alignment. */
if (!warn_strict_selector_match) if (!warn_strict_selector_match)
{ {
for (i = 0; i < TREE_VEC_LENGTH (method); i++) for (i = 0; i < (size_t) TREE_VEC_LENGTH (method); i++)
if (!comp_proto_with_proto (first_method, TREE_VEC_ELT (method, i), 0)) if (!comp_proto_with_proto (first_method, TREE_VEC_ELT (method, i), 0))
goto issue_warning; goto issue_warning;
@ -5103,7 +5103,7 @@ check_duplicates (tree method, int methods, int is_class)
identifier_to_locale (gen_method_decl (first_method))); identifier_to_locale (gen_method_decl (first_method)));
} }
for (i = 0; i < TREE_VEC_LENGTH (method); i++) for (i = 0; i < (size_t) TREE_VEC_LENGTH (method); i++)
{ {
bool type = TREE_CODE (TREE_VEC_ELT (method, i)) == INSTANCE_METHOD_DECL; bool type = TREE_CODE (TREE_VEC_ELT (method, i)) == INSTANCE_METHOD_DECL;
@ -5825,7 +5825,7 @@ insert_method_into_method_map (bool class_method, tree method)
/* Check all the existing prototypes. If any matches the /* Check all the existing prototypes. If any matches the
one we need to add, there is nothing to do because it's one we need to add, there is nothing to do because it's
already there. */ already there. */
for (i = 0; i < TREE_VEC_LENGTH (existing_entry); i++) for (i = 0; i < (size_t) TREE_VEC_LENGTH (existing_entry); i++)
if (comp_proto_with_proto (method, TREE_VEC_ELT (existing_entry, i), 1)) if (comp_proto_with_proto (method, TREE_VEC_ELT (existing_entry, i), 1))
return; return;
@ -5837,7 +5837,7 @@ insert_method_into_method_map (bool class_method, tree method)
new_entry = make_tree_vec (TREE_VEC_LENGTH (existing_entry) + 1); new_entry = make_tree_vec (TREE_VEC_LENGTH (existing_entry) + 1);
/* Copy the methods from the existing vector. */ /* Copy the methods from the existing vector. */
for (i = 0; i < TREE_VEC_LENGTH (existing_entry); i++) for (i = 0; i < (size_t) TREE_VEC_LENGTH (existing_entry); i++)
TREE_VEC_ELT (new_entry, i) = TREE_VEC_ELT (existing_entry, i); TREE_VEC_ELT (new_entry, i) = TREE_VEC_ELT (existing_entry, i);
/* Add the new method at the end. */ /* Add the new method at the end. */