svn commit: r581461 - /incubator/xap/trunk/codebase/src/xap/session/DeclarativeArgumentParser.js

[jm...@apache.org]

Author: jmargaris
Date: Tue Oct  2 18:21:12 2007
New Revision: 581461

URL: http://svn.apache.org/viewvc?rev=581461&view=rev
Log:
removed useless logging

Modified:
    incubator/xap/trunk/codebase/src/xap/session/DeclarativeArgumentParser.js

Modified: incubator/xap/trunk/codebase/src/xap/session/DeclarativeArgumentParser.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/codebase/src/xap/session/DeclarativeArgumentParser.js?rev=581461&r1=581460&r2=581461&view=diff
==============================================================================
--- incubator/xap/trunk/codebase/src/xap/session/DeclarativeArgumentParser.js (original)
+++ incubator/xap/trunk/codebase/src/xap/session/DeclarativeArgumentParser.js Tue Oct  2 18:21:12 2007
@@ -159,10 +159,7 @@
 
 xap.session.DeclarativeArgumentParser.prototype.parseArgument = function( args, startIndex,
 	sourceElement, clientEvent) {
-	
-	if (this._log.isTrace())
-		this._log.trace("Parse argument [ " + args + "] at start index:" + startIndex);
-		
+			
 	var arg = new String();
 	var parseResult = null;
 
@@ -318,8 +315,6 @@
  * @private
  */
 xap.session.DeclarativeArgumentParser.prototype._parseArguments = function( argumentsString, sourceElement, index, clientEvent){
-	if (this._log.isTrace())
-		this._log.trace("Parse arguments: " + argumentsString + " at index " + index);
 	var args = new Array();
 	//keep calling parseAgument until we are done,
 	//then stick all the results in an object[]
@@ -329,8 +324,6 @@
 		//why do we do +1 here? because if the last terminating character
 		//was , we need to start our search one AFTER the comma!
 		var result = this.parseArgument(argumentsString,argumentStart+1,sourceElement, clientEvent);
-		if (this._log.isTrace())
-			this._log.trace("Got back result " + result._resultObject);
 		if (result._resultObject!=null){
 			args.push(result._resultObject);
 			argumentStart = result._terminatingIndex;
@@ -349,8 +342,6 @@
 			break;
 		}
 	}
-	if (this._log.isTrace())
-		this._log.trace("Return parse result: " + args);
 	return new this._ParseResult(args,argumentStart);
 }
 
@@ -498,10 +489,7 @@
 		throw new xap.util.Exception("Could not find method: " + methodName);
     	
 	}
-	
-	if (this._log.isTrace())
-		xap.session.DeclarativeArgumentParser.s_log.debug("Fire object event [" + methodName +"] on the object " + mcoName + 
-			" with arguments " + args);
+
 	try{
 		return o[methodName].apply(o,args);
 	}
@@ -525,8 +513,6 @@
  
 xap.session.DeclarativeArgumentParser.prototype._parseString = function(args, startIndex, terminatingCharacter){
 	
-	if (this._log.isTrace())
-		this._log.trace("Parse string :" + args + " at start index " + startIndex);
 	var result = new this._ParseResult(null,startIndex);
 	var arg = new String();
 	startIndex++; //first must be a " or '