dojo.lang.shallowCopy has an unnessary if statement

[rfa@…]

Remove if(dojo.lang.isUndefined(ret[key]))

dojo.lang.shallowCopy = function(obj) {
	var ret = {}, key;
	for(key in obj) {ret[key] = obj[key];}
	return ret;
}
 REPLACES:
dojo.lang.shallowCopy = function(obj) {
	var ret = {}, key;
	for(key in obj) {
		if(dojo.lang.isUndefined(ret[key])) {
			ret[key] = obj[key];
		}
	}
	return ret;
}

[Tom Trenka]

There are a few situations (rare but there) where the isUndefined is actually necessary. Thanks for the thought though.