From db8d6e766a4895aa30211af3037a55a4a2c25d4f Mon Sep 17 00:00:00 2001
From: Martin Stubenschrott <stubenschrott@gmx.net>
Date: Fri, 12 Oct 2007 01:51:32 +0000
Subject: [PATCH] removed useless outer loop

---
 content/bookmarks.js | 53 +++++++++++++++++++++-----------------------
 1 file changed, 25 insertions(+), 28 deletions(-)

diff --git a/content/bookmarks.js b/content/bookmarks.js
index e91dd724..4d618f70 100644
--- a/content/bookmarks.js
+++ b/content/bookmarks.js
@@ -265,40 +265,37 @@ function Bookmarks() //{{{
             }
 
             var title, url, tags, keyword, extra;
+            var list = ":" + vimperator.util.escapeHTML(vimperator.commandline.getCommand()) + "<br/>" +
+                       "<table><tr align=\"left\" class=\"hl-Title\"><th>title</th><th>URL</th></tr>";
             for (var i = 0; i < items.length; i++)
             {
-                var list = ":" + vimperator.util.escapeHTML(vimperator.commandline.getCommand()) + "<br/>" +
-                           "<table><tr align=\"left\" class=\"hl-Title\"><th>title</th><th>URL</th></tr>";
-                for (var i = 0; i < items.length; i++)
+                title = vimperator.util.escapeHTML(items[i][1]);
+                if (title.length > 50)
+                    title = title.substr(0, 47) + "...";
+                url = vimperator.util.escapeHTML(items[i][0]);
+                keyword = items[i][2];
+                tags = items[i][3].join(", ");
+
+                extra = "";
+                if (keyword)
                 {
-                    title = vimperator.util.escapeHTML(items[i][1]);
-                    if (title.length > 50)
-                        title = title.substr(0, 47) + "...";
-                    url = vimperator.util.escapeHTML(items[i][0]);
-                    keyword = items[i][2];
-                    tags = items[i][3].join(", ");
-
-                    extra = "";
-                    if (keyword)
-                    {
-                        extra = "<span style=\"color: gray;\"> (keyword: <span style=\"color: red;\">" + vimperator.util.escapeHTML(keyword) + "</span>";
-                        if (tags)
-                            extra += " tags: <span style=\"color: blue;\">" + vimperator.util.escapeHTML(tags) + ")</span>";
-                        else
-                            extra += ")</span>";
-                    }
-                    else if (tags)
-                    {
-                        extra = "<span style=\"color: gray;\"> (tags: <span style=\"color: blue;\">" + vimperator.util.escapeHTML(tags) + "</span>)</span>";
-                    }
-
-
-                    list += "<tr><td>" + title + "</td><td style=\"color: green; width: 100%\">" + url + extra + "</td></tr>";
+                    extra = "<span style=\"color: gray;\"> (keyword: <span style=\"color: red;\">" + vimperator.util.escapeHTML(keyword) + "</span>";
+                    if (tags)
+                        extra += " tags: <span style=\"color: blue;\">" + vimperator.util.escapeHTML(tags) + ")</span>";
+                    else
+                        extra += ")</span>";
+                }
+                else if (tags)
+                {
+                    extra = "<span style=\"color: gray;\"> (tags: <span style=\"color: blue;\">" + vimperator.util.escapeHTML(tags) + "</span>)</span>";
                 }
-                list += "</table>";
 
-                vimperator.commandline.echo(list, vimperator.commandline.HL_NORMAL, vimperator.commandline.FORCE_MULTILINE);
+
+                list += "<tr><td>" + title + "</td><td style=\"color: green; width: 100%\">" + url + extra + "</td></tr>";
             }
+            list += "</table>";
+
+            vimperator.commandline.echo(list, vimperator.commandline.HL_NORMAL, vimperator.commandline.FORCE_MULTILINE);
         }
     }