From 47e3fe106a9cd7ae72272955194a0ea9ae528820 Mon Sep 17 00:00:00 2001 From: Michael Drake Date: Wed, 19 Sep 2012 14:30:47 +0100 Subject: Fix comparison of signed/unsigned variables. --- .gitignore | 1 - include/dom/html/html_select_element.h | 2 +- src/html/html_select_element.c | 4 ++-- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/.gitignore b/.gitignore index a12a172..9d29615 100644 --- a/.gitignore +++ b/.gitignore @@ -3,5 +3,4 @@ build-* build/docs examples/dom-structure-dump test/level*.c -test/INDEX *~ diff --git a/include/dom/html/html_select_element.h b/include/dom/html/html_select_element.h index 6999368..42aef00 100644 --- a/include/dom/html/html_select_element.h +++ b/include/dom/html/html_select_element.h @@ -80,7 +80,7 @@ dom_exception dom__html_select_element_add(dom_html_select_element *select, (struct dom_html_element *) (e), \ (struct dom_html_element *) (b)) dom_exception dom_html_select_element_remove(dom_html_select_element *ele, - long index); + int32_t index); dom_exception dom_html_select_element_blur(struct dom_html_select_element *ele); dom_exception dom_html_select_element_focus(struct dom_html_select_element *ele); diff --git a/src/html/html_select_element.c b/src/html/html_select_element.c index 4efffb9..12ea6ba 100644 --- a/src/html/html_select_element.c +++ b/src/html/html_select_element.c @@ -536,7 +536,7 @@ dom_exception dom__html_select_element_add(dom_html_select_element *select, } dom_exception dom_html_select_element_remove(dom_html_select_element *ele, - long index) + int32_t index) { dom_exception err; uint32_t len; @@ -547,7 +547,7 @@ dom_exception dom_html_select_element_remove(dom_html_select_element *ele, return err; /* Ensure index is in range */ - if (index >= len) + if (index < 0 || (uint32_t)index >= len) return DOM_NO_ERR; err = dom_html_options_collection_item(ele->options, index, &option); -- cgit v1.2.3