Skip properties inherited from array.prototype

Alex Kwiatkowski & Daniel Yankowsky · Alex Kwiatkowski & Daniel Yankowsky · commit 00dbce612fcf · 2012-07-11T11:45:28.000-04:00
diff --git a/src/fonts.js b/src/fonts.js
@@ -2773,14 +2773,16 @@ var Font = (function FontClosure() {
             }
           }
           for (var index in newGlyphUnicodes) {
-            var unicode = newGlyphUnicodes[index];
-            if (reverseMap[unicode]) {
-              // avoiding assigning to the same unicode
-              glyphs[index].unicode = unusedUnicode++;
-              continue;
+            if (newGlyphUnicodes.hasOwnProperty(index)) {
+              var unicode = newGlyphUnicodes[index];
+              if (reverseMap[unicode]) {
+                // avoiding assigning to the same unicode
+                glyphs[index].unicode = unusedUnicode++;
+                continue;
+              }
+              glyphs[index].unicode = unicode;
+              reverseMap[unicode] = index;
             }
-            glyphs[index].unicode = unicode;
-            reverseMap[unicode] = index;
           }
           this.useToFontChar = true;
         }

Original file line number	Diff line number	Diff line change
`@@ -2773,14 +2773,16 @@ var Font = (function FontClosure() {`
`2773`	`2773`	`}`
`2774`	`2774`	`}`
`2775`	`2775`	`for (var index in newGlyphUnicodes) {`
`2776`		`- var unicode = newGlyphUnicodes[index];`
`2777`		`- if (reverseMap[unicode]) {`
`2778`		`- // avoiding assigning to the same unicode`
`2779`		`- glyphs[index].unicode = unusedUnicode++;`
`2780`		`- continue;`
	`2776`	`+ if (newGlyphUnicodes.hasOwnProperty(index)) {`
	`2777`	`+ var unicode = newGlyphUnicodes[index];`
	`2778`	`+ if (reverseMap[unicode]) {`
	`2779`	`+ // avoiding assigning to the same unicode`
	`2780`	`+ glyphs[index].unicode = unusedUnicode++;`
	`2781`	`+ continue;`
	`2782`	`+ }`
	`2783`	`+ glyphs[index].unicode = unicode;`
	`2784`	`+ reverseMap[unicode] = index;`
`2781`	`2785`	`}`
`2782`		`- glyphs[index].unicode = unicode;`
`2783`		`- reverseMap[unicode] = index;`
`2784`	`2786`	`}`
`2785`	`2787`	`this.useToFontChar = true;`
`2786`	`2788`	`}`