fix #685 by not caching dep subscriber length

yyx990803 · yyx990803 · commit 229f0b2058d3 · 2015-01-26T13:47:09.000-05:00
diff --git a/src/observer/dep.js b/src/observer/dep.js
@@ -42,8 +42,8 @@ p.removeSub = function (sub) {
  */
 
 p.notify = function () {
-  for (var i = 0, l = this.subs.length; i < l; i++) {
-    this.subs[i].update()
+  for (var i = 0, subs = this.subs; i < subs.length; i++) {
+    subs[i].update()
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -42,8 +42,8 @@ p.removeSub = function (sub) {`
`42`	`42`	`*/`
`43`	`43`
`44`	`44`	`p.notify = function () {`
`45`		`- for (var i = 0, l = this.subs.length; i < l; i++) {`
`46`		`- this.subs[i].update()`
	`45`	`+ for (var i = 0, subs = this.subs; i < subs.length; i++) {`
	`46`	`+ subs[i].update()`
`47`	`47`	`}`
`48`	`48`	`}`
`49`	`49`