From b21460d5caeb10f89d983301a9d6086f1cd28c38 Mon Sep 17 00:00:00 2001
From: Dmitry Prokashev <heliosframework@gmail.com>
Date: Sun, 12 Apr 2015 04:37:27 +0200
Subject: [PATCH] cleanup

---
 README.md   |  5 ++++-
 viewport.js | 39 ++++++++++++++++++---------------------
 2 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/README.md b/README.md
index 42f5448..93b83d1 100644
--- a/README.md
+++ b/README.md
@@ -23,7 +23,7 @@ following advantages:
 - it is written on vanilla javascript, does not have dependencies and
   works anywhere;
 
-- its size is only 1581 bytes minified;
+- its size is only 1566 bytes minified;
 
 - it has a simple and flexible API which shows:
 
@@ -152,6 +152,9 @@ viewport):
 var currentSection = document.getElementById('myViewport').currentSection;
 ```
 
+If you change / create the sections dynamically after the page
+load, invoke `viewport.reset()` to update the listeners.
+
 If you create a navigation panel reflecting the scrolling state,
 replace the scrollbars with [intence](http://asvd.github.io/intence)
 indicator: it designates a scrollable area in more clear and intuitive
diff --git a/viewport.js b/viewport.js
index f2f10c4..c03372f 100644
--- a/viewport.js
+++ b/viewport.js
@@ -52,29 +52,29 @@
       
       
     var reset = function() {
-        var i, j, isBody, hasViewportClass, classes,
-            listener, section, viewport, scroller, entry, sections;
+        var i=0, j, isBody, hasViewportClass, classes,
+            listener, section, viewport, scroller, entry=_null, sections;
 
         // running through existing entries and removing listeners
-        for (i = 0; i < entries[length]; i++) {
+        for (;i < entries[length];) {
             listener = entries[i].r.vpl;
-            entries[i].r[removeEventListener](scroll, listener, 0);
+            entries[i++].r[removeEventListener](scroll, listener, 0);
             _window[removeEventListener](resize, listener, 0);
         }
 
         // rebuilding entries
         entries = [];
         sections = _document.getElementsByClassName('section');
-        for (i = 0; i < sections[length]; i++) {
+        for (i = 0; i < sections[length];) {
             // searching for a parent viewport
-            viewport = section = sections[i];
-            do {
-                hasViewportClass = 0;
+            viewport = section = sections[i++];
+            while(1) {
+                hasViewportClass = j = 0;
                 isBody = viewport == _document.body;
                 if (!isBody) {
                     classes = viewport.className.split(' ');
-                    for (j = 0; j < classes[length]; j++) {
-                        if (classes[j] == VIEWPORT) {
+                    for (;j < classes[length];) {
+                        if (classes[j++] == VIEWPORT) {
                             hasViewportClass = 1;
                             break;
                         }
@@ -86,20 +86,18 @@
                 }
 
                 viewport = viewport.parentNode;
-            } while(1);
+            }
 
             // searching for exisiting entry for the viewport
-            entry = _null;
             for (j = 0; j < entries[length]; j++) {
                 if (entries[j].v == viewport) {
                     entry = entries[j];
-                    break;
                 }
             }
 
+            // creating a new entry if not found
             if (!entry) {
                 scroller = viewport.scroller||viewport;
-                // a new entry for the viewport
                 entry = {
                     v : viewport,
                     r : scroller,
@@ -107,8 +105,7 @@
                 };
 
                 // listener invoked upon the viewport scroll
-                scroller.vpl = (function(entry) {
-                return function() {
+                scroller.vpl = (function(entry) {return function() {
                     var scroller = entry.r;
                     var vRect = scroller[getBoundingClientRect]();
 
@@ -137,8 +134,8 @@
                     // (and searching for the closest section)
                     var closest = _null;
                     var minDist = _null;
-                    for (var i = 0; i < entry.s[length]; i++) {
-                        var section = entry.s[i];
+                    for (var i = 0; i < entry.s[length];) {
+                        var section = entry.s[i++];
 
                         var sRect = section[getBoundingClientRect]();
                         var sTop = sRect[top];
@@ -155,7 +152,7 @@
 
                         // viewport to section distance, normalized
                         var vVertDist =
-                            Math_max( 0, Math_abs(vTopLocation - 0.5) - 0.5);
+                            Math_max(0, Math_abs(vTopLocation - 0.5) - 0.5);
                         var vHorizDist =
                             Math_max(0, Math_abs(vLeftLocation - 0.5) - 0.5);
 
@@ -223,8 +220,8 @@
         }
 
         // initially setting-up the properties
-        for (i = 0; i < entries[length]; i++) {
-            entry.r.vpl();
+        for (i = 0; i < entries[length];) {
+            entries[i++].r.vpl();
         }
     }