dojox.lang.aspect: cleanup. !strict

git-svn-id: http://svn.dojotoolkit.org/src/dojox/trunk@13712 560b804f-0ae3-0310-86f3-f6aa0a117693

lang/README

@@ -12,20 +12,25 @@ Credits
 -------------------------------------------------------------------------------
 Project description
 
-Implementation of common functional operations, and provisions
-Later we can add other JS language-related helpers.
+Implementation of common functional operations, and provisions, aspect-oriented
+helpers. Later we can add other JS language-related helpers.
 -------------------------------------------------------------------------------
 Dependencies:
 
 None.
 -------------------------------------------------------------------------------
 Documentation
 
+For now:
+
+dojox.lang.functional:
+http://lazutkin.com/blog/2008/jan/12/functional-fun-javascript-dojo/
+
 -------------------------------------------------------------------------------
 Installation instructions
 
 Grab the following from the Dojo SVN Repository:
-http://svn.dojotoolkit.org/var/src/dojo/dojox/trunk/lang/*
+http://svn.dojotoolkit.org/src/dojo/dojox/trunk/lang/*
 
 Install into the following directory structure:
 /dojox/lang/
@@ -35,3 +40,6 @@ Install into the following directory structure:
 Additional Notes
 
 See tests and the source for more details.
+
+LICENSE in this directory contains the MIT license by Oliver Steele for
+dojox.lang.functional.lambda, which was derived from his original implementation.

lang/aspect/memoizer.js

@@ -3,17 +3,6 @@ dojo.provide("dojox.lang.aspect.memoizer");
 (function(){
 	var aop = dojox.lang.aspect;
 
-	var reset = function(/*String|Array?*/ method){
-		var that = aop.getContext().instance, t;
-		if(!(t = that.__memoizerCache)){ return; }
-		if(arguments.length == 0){
-			delete that.__memoizerCache;
-		}else if(dojo.isArray(method)){
-			dojo.forEach(method, function(m){ delete t[m]; });
-		}else{
-			delete t[method];
-		}
-	};
 	var memoize1 = {
 		around: function(key){
 			var ctx = aop.getContext(), self = ctx.joinPoint, that = ctx.instance, t, u, ret;
@@ -24,8 +13,7 @@ dojo.provide("dojox.lang.aspect.memoizer");
 			if(!(t = that.__memoizerCache)){ t = that.__memoizerCache = {}; }
 			if(!(u = t[self.targetName])){ u = t[self.targetName] = {}; }
 			return u[key] = ret;
-		},
-		reset: reset
+		}
 	};
 
 	var memoizeN = function(/*Function*/keyMaker){
@@ -40,8 +28,7 @@ dojo.provide("dojox.lang.aspect.memoizer");
 				if(!(t = that.__memoizerCache)){ t = that.__memoizerCache = {}; }
 				if(!(u = t[self.targetName])){ u = t[self.targetName] = {}; }
 				return u[key] = ret;
-			},
-			reset: reset
+			}
 		};
 	};
 
@@ -55,5 +42,4 @@ dojo.provide("dojox.lang.aspect.memoizer");
 
 		return arguments.length == 0 ? memoize1 : memoizeN(keyMaker);	// Object
 	};
-	aop.memoizer._reset = reset;
 })();

lang/aspect/memoizerGuard.js

@@ -1,12 +1,23 @@
 dojo.provide("dojox.lang.aspect.memoizerGuard");
-dojo.require("dojox.lang.aspect.memoizer");
 
 (function(){
-	var aop = dojox.lang.aspect, reset = aop.memoizer._reset;
+	var aop = dojox.lang.aspect,
+		reset = function(/*String|Array?*/ method){
+			var that = aop.getContext().instance, t;
+			if(!(t = that.__memoizerCache)){ return; }
+			if(arguments.length == 0){
+				delete that.__memoizerCache;
+			}else if(dojo.isArray(method)){
+				dojo.forEach(method, function(m){ delete t[m]; });
+			}else{
+				delete t[method];
+			}
+		};
+
 
 	aop.memoizerGuard = function(/*String|Array?*/ method){
 		// summary:
-		//		Invalidates cache (see dojox.lang.aspect.memoizer)
+		//		Invalidates the memoizer's cache (see dojox.lang.aspect.memoizer)
 		//		after calling certain methods.
 		//
 		// method:

lang/aspect/tracer.js

@@ -31,7 +31,7 @@ dojo.provide("dojox.lang.aspect.tracer");
 		}
 	});
 
-	aop.tracer = function(/*Boolean*/ grouping, /*Boolean*/ suppressInstance){
+	aop.tracer = function(/*Boolean*/ grouping){
 		// summary:
 		//		Returns an object, which can be used to trace calls with Firebug's console.
 		//		Prints argument, a return value, or an exception.