From e4c9e95f3b241dc113b790add029f6ab87b2320b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jonatan=20P=C3=A5lsson?= Date: Tue, 8 Apr 2014 14:01:47 +0200 Subject: Fixed duplication in select lists --- browser/browserview.h | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/browser/browserview.h b/browser/browserview.h index 0ec7765..0cff39d 100644 --- a/browser/browserview.h +++ b/browser/browserview.h @@ -65,18 +65,20 @@ public slots: void handleSelectElement(const QWebElement &elem) { conn::brw::SelectableOptionList options; - QList elems; QWebElement first = elem.firstChild(); QWebElement last = elem.lastChild(); + QWebElement iter = first; - conn::brw::SelectableOption o; - o.strValue = first.toPlainText(); - options.append(o); - elems.append(first); - while (elems.last() != last) { + while (iter != last) { conn::brw::SelectableOption o; - o.strValue = elems.last().toPlainText(); - elems.append(elems.last().nextSibling()); + o.strValue = iter.toPlainText(); + options.append(o); + iter = iter.nextSibling(); + } + + if (iter == last) { + conn::brw::SelectableOption o; + o.strValue = iter.toPlainText(); options.append(o); } -- cgit v1.2.1