From 110cffc14ca58ff7b09fb271a66eb573688aec87 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BCrg=20Lehni?= <juerg@scratchdisk.com>
Date: Mon, 4 Jul 2011 23:28:28 +0200
Subject: [PATCH] Improve comments for bounds functions.

---
 src/item/PlacedItem.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/item/PlacedItem.js b/src/item/PlacedItem.js
index d77ea412..151024d7 100644
--- a/src/item/PlacedItem.js
+++ b/src/item/PlacedItem.js
@@ -60,11 +60,11 @@ var PlacedItem = this.PlacedItem = Item.extend(/** @lends PlacedItem# */{
 	},
 
 	getBounds: function(/* matrix */) {
-		// The bounds of PlacedItems are the same as the strokeBounds, but are
-		// wrapped in a LinkedRectangle that catch changes for us.
 		var useCache = arguments[0] === undefined;
 		if (useCache && this._bounds)
 			return this._bounds;
+		// The bounds of PlacedItems are the same as the strokeBounds, but are
+		// wrapped in a LinkedRectangle that catch changes for us.
 		var bounds = this.getStrokeBounds(arguments[0]);
 		if (useCache)
 			bounds = this._bounds = this._createBounds(bounds);
@@ -76,11 +76,11 @@ var PlacedItem = this.PlacedItem = Item.extend(/** @lends PlacedItem# */{
 			useCache = matrix === undefined;
 		if (useCache && this[cacheName])
 			return this[cacheName];
+		// Concatenate the passed matrix with the internal one
 		matrix = matrix ? matrix.clone().concatenate(this._matrix)
 				: this._matrix;
 		// Call _calculateBounds, which needs to be defined in the subclasses:
 		var bounds = this._calculateBounds(getter, matrix);
-		// TODO: Clear cache
 		if (useCache)
 			this[cacheName] = bounds;
 		return bounds;