From 932f14bd6e8730c544929c6a2cde24b141430b07 Mon Sep 17 00:00:00 2001
From: gorhill <rhill@raymondhill.net>
Date: Fri, 6 Feb 2015 06:49:48 -0500
Subject: [PATCH] code review re. #695

---
 src/js/storage.js | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/src/js/storage.js b/src/js/storage.js
index eb666ed8f..ff6c267e4 100644
--- a/src/js/storage.js
+++ b/src/js/storage.js
@@ -307,25 +307,19 @@
 
         // Load all filter lists which are not disabled
         var location;
-        var filterlistLoadingCount = 0;
-        while ( filterlistCount !== 0 ) {
-            location = locations.pop()
-            if ( !location ) {
-                break;
-            }
+        while ( location = locations.pop() ) {
             // rhill 2013-12-09:
             // Ignore list if disabled
             // https://github.com/gorhill/httpswitchboard/issues/78
-            if ( lists[location].off === false ) {
-                filterlistLoadingCount += 1;
-                µb.assets.get(location, mergeBlacklist);
+            if ( lists[location].off ) {
+                filterlistCount -= 1;
                 continue;
             }
-            filterlistCount -= 1;
+            µb.assets.get(location, mergeBlacklist);
         }
         // https://github.com/gorhill/uBlock/issues/695
         // It may happen not a single filter list is selected
-        if ( filterlistLoadingCount === 0 ) {
+        if ( filterlistCount === 0 ) {
             loadBlacklistsEnd();
         }
     };