From 2a5f885f5039ef6bcfef2e03bbc164fea55e99b5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=BCrg=20Lehni?= Date: Wed, 1 Jun 2011 09:57:18 +0100 Subject: [PATCH] "for var in" is good practise in such a case, as we don't need additional functionality of Base.each(). --- src/project/Project.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/project/Project.js b/src/project/Project.js index 7c17323a..589451eb 100644 --- a/src/project/Project.js +++ b/src/project/Project.js @@ -164,8 +164,6 @@ var Project = this.Project = Base.extend({ * Deselects all selected items in the project. */ deselectAll: function() { - // TODO: is using for var i in good practice? - // or should we use Base.each? (JP) for (var i in this._selectedItems) this._selectedItems[i].setSelected(false); },