[Pkg-mozext-commits] [firebug] 23/59: Simplify ArrBase.hasSpecialProperties

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository firebug.

commit 79998b6b7cf47c1fede2c85897a4458728e7a4fb
Author: Simon Lindholm <simon.lindholm10 at gmail.com>
Date:   Tue Jul 29 19:07:15 2014 +0200

    Simplify ArrBase.hasSpecialProperties
---
 extension/content/firebug/chrome/reps.js | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/extension/content/firebug/chrome/reps.js b/extension/content/firebug/chrome/reps.js
index fe5173e..4ca4d83 100644
--- a/extension/content/firebug/chrome/reps.js
+++ b/extension/content/firebug/chrome/reps.js
@@ -681,19 +681,14 @@ FirebugReps.ArrBase = domplate(FirebugReps.Obj,
            return x === y.toString();
         }
 
-        var n = 0;
         var obj = Wrapper.unwrapObject(array);
         var props = Object.getOwnPropertyNames(obj);
         for (var i=0; i<props.length; i++)
         {
             var p = props[i];
 
-            // Valid indexes are skipped
-            if (isInteger(p))
-                continue;
-
-            // Ignore standard 'length' property, anything else is custom.
-            if (p != "length")
+            // Valid indices and 'length' are skipped, anything else is custom.
+            if (!isInteger(p) && p != "length")
                 return true;
         }
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/firebug.git