You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by jb...@apache.org on 2010/08/17 18:44:48 UTC
svn commit: r986378 - in
/commons/sandbox/gsoc/2010/scxml-js/trunk/src/javascript/scxml/cgf: util.js
util/commandLine.js
Author: jbeard
Date: Tue Aug 17 16:44:47 2010
New Revision: 986378
URL: http://svn.apache.org/viewvc?rev=986378&view=rev
Log:
Refactoring: removed util.js and its references, as last remaining method toJsArray, could be expressed more succintly using new js idiom Array.prototype.slice.call
Removed:
commons/sandbox/gsoc/2010/scxml-js/trunk/src/javascript/scxml/cgf/util.js
Modified:
commons/sandbox/gsoc/2010/scxml-js/trunk/src/javascript/scxml/cgf/util/commandLine.js
Modified: commons/sandbox/gsoc/2010/scxml-js/trunk/src/javascript/scxml/cgf/util/commandLine.js
URL: http://svn.apache.org/viewvc/commons/sandbox/gsoc/2010/scxml-js/trunk/src/javascript/scxml/cgf/util/commandLine.js?rev=986378&r1=986377&r2=986378&view=diff
==============================================================================
--- commons/sandbox/gsoc/2010/scxml-js/trunk/src/javascript/scxml/cgf/util/commandLine.js (original)
+++ commons/sandbox/gsoc/2010/scxml-js/trunk/src/javascript/scxml/cgf/util/commandLine.js Tue Aug 17 16:44:47 2010
@@ -23,8 +23,7 @@ native Java objects back to JavaScript o
require.def(
"src/javascript/scxml/cgf/util/commandLine",
- ["src/javascript/scxml/cgf/util"],
- function(util){
+ function(){
return {
parseCommandLine: function(optionsMap,args){
@@ -52,7 +51,7 @@ require.def(
line = parser.parse( options, args );
lineMap = this.lineToJsMap(line);
- lineMap.args = util.toJsArray(line.getArgs());
+ lineMap.args = Array.prototype.slice.call(line.getArgs());
/*
for (var optName in lineMap){