Fix/mute issues found by Coverity scan

This makes the code free of Coverity scan issues.
It is sometimes quite pedantic and expects/suggests some
coding habits, thus certain changes may look weird, but for a good
thing, I hope. The code is also tagged with Coverity scan
suppressions, to keep the code as is and hide the warning too.
Also note that Coverity treats g_return_if_fail(), g_assert() and
similar macros as unreliable, and it's true these can be disabled
during the compile time, thus it brings in other set of 'weird'
changes.

1 files changed, 3 insertions, 5 deletions

diff --git a/modules/itip-formatter/e-mail-part-itip.c b/modules/itip-formatter/e-mail-part-itip.c
index e4e6302822..447ff0d13b 100644
--- a/modules/itip-formatter/e-mail-part-itip.c
+++ b/modules/itip-formatter/e-mail-part-itip.c
@@ -88,16 +88,14 @@ mail_part_itip_bind_dom_element (EMailPart *part,
 
 	if (!WEBKIT_DOM_IS_HTML_IFRAME_ELEMENT (element)) {
 		WebKitDOMNodeList *nodes;
-		guint ii, length;
+		guint length;
 
 		nodes = webkit_dom_element_get_elements_by_tag_name (
 			element, "iframe");
 		length = webkit_dom_node_list_get_length (nodes);
-		for (ii = 0; ii < length; ii++) {
+		if (length > 0)
 			element = WEBKIT_DOM_ELEMENT (
-				webkit_dom_node_list_item (nodes, ii));
-			break;
-		}
+				webkit_dom_node_list_item (nodes, 0));
 	}
 
 	g_return_if_fail (WEBKIT_DOM_IS_HTML_IFRAME_ELEMENT (element));