Author: reebalazs
Date: Mon Nov 10 08:02:38 2008
New Revision: 59845

Modified:
   kukit/kukit.js/trunk/kukit/actionreg.js
   kukit/kukit.js/trunk/kukit/dom.js
   kukit/kukit.js/trunk/kukit/errors.js
   kukit/kukit.js/trunk/kukit/forms.js
   kukit/kukit.js/trunk/kukit/kssparser.js
   kukit/kukit.js/trunk/kukit/oper.js
   kukit/kukit.js/trunk/kukit/plugin.js
   kukit/kukit.js/trunk/kukit/resourcedata.js
   kukit/kukit.js/trunk/kukit/serveraction.js
   kukit/kukit.js/trunk/kukit/tokenizer.js
   kukit/kukit.js/trunk/kukit/utils.js
Log:
Fix missing semicolons
make default logging of xmlhttprequest readystate less verbose
fix deprecations in plugin.js

Modified: kukit/kukit.js/trunk/kukit/actionreg.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/actionreg.js     (original)
+++ kukit/kukit.js/trunk/kukit/actionreg.js     Mon Nov 10 08:02:38 2008
@@ -35,7 +35,7 @@
 ;;; if (typeof(func) == 'undefined') {
 ;;;     kukit.e = '[func] argument is mandatory when registering an action';
 ;;;     kukit.e += ' [actionregistry.register].';
-;;;     throw new error(kukit.e);
+;;;     throw new Error(kukit.e);
 ;;; }
     if (this.content[name]) {
         // do not allow redefinition

Modified: kukit/kukit.js/trunk/kukit/dom.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/dom.js   (original)
+++ kukit/kukit.js/trunk/kukit/dom.js   Mon Nov 10 08:02:38 2008
@@ -407,7 +407,7 @@
     function(node, attrname, recurseParents, getter) {
     return dom.locateMarkup(node,
             recurseParents, getter, attrname).value;
-}
+};
 
 /*
 * From http://xkr.us/articles/dom/iframe-document/

Modified: kukit/kukit.js/trunk/kukit/errors.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/errors.js        (original)
+++ kukit/kukit.js/trunk/kukit/errors.js        Mon Nov 10 08:02:38 2008
@@ -146,8 +146,8 @@
 ;;; };
 
 
-;;; err.parsingError = function(message, cursor){
-;;;    var kw = {}
+err.parsingError = function(message, cursor){
+   var kw = {};
 ;;;    if (cursor) {
 ;;;         kw.errpos = cursor.pos;
 ;;;         kw.errrow = cursor.row;
@@ -158,8 +158,8 @@
 ;;;         kw.errrow = null;
 ;;;         kw.errcol = null;
 ;;;    }
-;;;    return setErrorInfo(null, 'ParsingError', message, kw);
-;;; };
+   return setErrorInfo(null, 'ParsingError', message, kw);
+};
 
 
 /* Exceptions that re-throw (annotate) an already caught error */
@@ -186,7 +186,7 @@
 
 
 ;;; err.kssParsingError = function(e, url){
-;;;    var kw = {url: url}
+;;;    var kw = {url: url};
 ;;;    var message = 'Error parsing KSS at ' + url;
 ;;;    return setErrorInfo(e, 'KssParsingError', message, kw);
 ;;; };

Modified: kukit/kukit.js/trunk/kukit/forms.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/forms.js (original)
+++ kukit/kukit.js/trunk/kukit/forms.js Mon Nov 10 08:02:38 2008
@@ -132,7 +132,7 @@
 
 this.initialize = function(target) {
     this.target = target;
-}
+};
 
 this.queryForm = function() {
     // Find the form that contains the target node.

Modified: kukit/kukit.js/trunk/kukit/kssparser.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/kssparser.js     (original)
+++ kukit/kukit.js/trunk/kukit/kssparser.js     Mon Nov 10 08:02:38 2008
@@ -63,23 +63,23 @@
 };
 
 var _returnComment = function() {
-    return new kssp.Comment(this.cursor, kssp.openComment)
+    return new kssp.Comment(this.cursor, kssp.openComment);
 };
 
 var _returnString = function() {
-    return new kssp.String(this.cursor, kssp.quote)
+    return new kssp.String(this.cursor, kssp.quote);
 };
 
 var _returnString2 = function() {
-    return new kssp.String2(this.cursor, kssp.dquote)
+    return new kssp.String2(this.cursor, kssp.dquote);
 };
 
 var _returnMethodArgs = function() {
-    return new kssp.MethodArgs(this.cursor, kssp.openParent)
+    return new kssp.MethodArgs(this.cursor, kssp.openParent);
 };
 
 var _returnBackslashed = function() {
-    return new kssp.Backslashed(this.cursor, kssp.backslash)
+    return new kssp.Backslashed(this.cursor, kssp.backslash);
 };
 
 /*
@@ -556,7 +556,7 @@
 
 this.process = function() {
     // For multivalue only
-    this.values = []
+    this.values = [];
     // Parse all tokens (including first and last)
     var context = {'nextTokenIndex': 0};
     this.txt = '';
@@ -955,7 +955,7 @@
     while (context.nextTokenIndex < this.result.length) {
         this.digestTxt(context, kukit.tk.Fraction, kssp.Comment,
             kssp.String, kssp.String2);
-        var cursor = new kukit.tk.Cursor(context.txt + ' ')
+        var cursor = new kukit.tk.Cursor(context.txt + ' ');
         var parser = new kssp.KssSelector(cursor, null, true);
         this.selectors.push(parser.kssSelector);
         if (context.nextTokenIndex == this.result.length) break;
@@ -972,10 +972,10 @@
 };
 kssp.KssSelectors = kukit.tk.mkParser('kssselectors', {
     "'": function() {
-             return new kssp.StringInSelector(this.cursor, kssp.quote)
+             return new kssp.StringInSelector(this.cursor, kssp.quote);
              },
     '"': function() {
-             return new kssp.String2InSelector(this.cursor, kssp.dquote)
+             return new kssp.String2InSelector(this.cursor, kssp.dquote);
              },
     ",": _mkReturnToken(kssp.comma),
     "{": _emitAndReturn,
@@ -1113,7 +1113,7 @@
 kssp.KssSelector = kukit.tk.mkParser('kssselector', {
     ":": function() {
              return [new kssp.colon(this.cursor),
-                     new kssp.EventValue(this.cursor)]
+                     new kssp.EventValue(this.cursor)];
              },
     "{": _emitAndReturn,
     "\/\*": _returnComment

Modified: kukit/kukit.js/trunk/kukit/oper.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/oper.js  (original)
+++ kukit/kukit.js/trunk/kukit/oper.js  Mon Nov 10 08:02:38 2008
@@ -289,14 +289,14 @@
     if (this.eventRule) {  
         // if it has actions, the answer is yes
         if (this.eventRule.actions.hasActions())
-            return true
+            return true;
         // if we have a default action, we will return true in any case
         // because we may want to call it.
         // The reason for this check is, that a default action is also
         // valid, even if it received no parms in the eventRule,
         // in which case it is not present as an action.
         var kssevent = kukit.eventsGlobalRegistry.get(
-            this.getEventNamespace(), this.getEventName())
+            this.getEventNamespace(), this.getEventName());
         var methodName = kssevent.defaultActionMethodName;
         return (typeof methodName != 'undefined');
     } else

Modified: kukit/kukit.js/trunk/kukit/plugin.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/plugin.js        (original)
+++ kukit/kukit.js/trunk/kukit/plugin.js        Mon Nov 10 08:02:38 2008
@@ -971,11 +971,11 @@
         }
     }
     if (parms.allnodes) {
-        binder.__continueEvent_allNodes__(parms.name,
+        binder.continueEventAllNodes(parms.name,
             actionParameters);
     } else {
         // execution happens on the orignode
-        binder.__continueEvent__(parms.name, oper.orignode,
+        binder.continueEvent(parms.name, oper.orignode,
             actionParameters);
     }
 });

Modified: kukit/kukit.js/trunk/kukit/resourcedata.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/resourcedata.js  (original)
+++ kukit/kukit.js/trunk/kukit/resourcedata.js  Mon Nov 10 08:02:38 2008
@@ -157,7 +157,7 @@
     //      key: value;
 this.initialize = function(txt) {    
     this.txt = txt;
-}
+};
 
 this.check = function(registry) {
     // use the IdentityPP provider.

Modified: kukit/kukit.js/trunk/kukit/serveraction.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/serveraction.js  (original)
+++ kukit/kukit.js/trunk/kukit/serveraction.js  Mon Nov 10 08:02:38 2008
@@ -135,8 +135,8 @@
 };
 
 this.notifyServer_done = function(domDoc) {
-;;; var msg = 'Request readyState = ' + domDoc.readyState + '.';
-;;; kukit.logDebug(msg);
+//;;; var msg = 'Request readyState = ' + domDoc.readyState + '.';
+//;;; kukit.logDebug(msg);
     if (domDoc.readyState == 4) {
         // notify the queue that we are done
         var success = this.oper.queueItem.receivedResult();
@@ -288,7 +288,8 @@
 ;;;             }
             throw kukit.err.responseParsingError(kukit.E);
         }
-    }
+    };
+
     if (dom == null) {
         // this should not happen
 ;;;     kukit.E = 'Neither xml nor html payload.';

Modified: kukit/kukit.js/trunk/kukit/tokenizer.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/tokenizer.js     (original)
+++ kukit/kukit.js/trunk/kukit/tokenizer.js     Mon Nov 10 08:02:38 2008
@@ -68,7 +68,7 @@
 };
 
 this.nextStep = function() {
-    var table = this.table
+    var table = this.table;
     var cursor = this.cursor;
     // Search for symbol according to table.
     var best_pos = cursor.text.length;
@@ -239,7 +239,7 @@
 tk.mkParser = function(symbol, table, _class) {
     // Poor man's subclassing.
     f = function(cursor, tokenClass, isTopLevelParser) {
-        this.table = table
+        this.table = table;
         this.cursor = cursor;
         this.startpos = cursor.pos;
         this.finished = false;

Modified: kukit/kukit.js/trunk/kukit/utils.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/utils.js (original)
+++ kukit/kukit.js/trunk/kukit/utils.js Mon Nov 10 08:02:38 2008
@@ -54,7 +54,7 @@
 try {
     kukit.HAVE_SAFARI = navigator.vendor && 
         navigator.vendor.indexOf('Apple') != -1;
-    kukit.HAVE_IE =  eval("_SARISSA_IS_IE");
+    kukit.HAVE_IE = eval("_SARISSA_IS_IE");
 } catch (e) {}
 
 ;;; // Activation of extra logging panel: if necessary
@@ -82,7 +82,7 @@
         if (c.indexOf(nameEQ) == 0) return c.substring(nameEQ.length, 
c.length);
     }
     return null;
-}
+};
 
 // a null function that is used for logging
 kukit._null = function() {};
@@ -162,7 +162,7 @@
 ;;;     kukit.dom.createCookie(kukit._COOKIE_LOGLEVEL, level);
 ;;;     // re-establish the log handlers, based on this cookie setting
 ;;;     kukit._logger.updateLogLevel();
-}
+};
 
 // We want a way of knowing if Firebug is available :
 // it is very convenient to log a node in Firebug;
@@ -192,7 +192,7 @@
 ;;;         this.logWarning = console.warn;
 ;;;         this.logError = console.error;
 ;;;         this.logFatal = console.error;
-;;;     }
+;;;     };
 ;;; }
 
 ;;; // check whether the logging stuff of MochiKit is available
@@ -211,7 +211,7 @@
 ;;;         this.logWarning = MochiKit.Logging.logWarning;
 ;;;         this.logError = MochiKit.Logging.logError;
 ;;;         this.logFatal = MochiKit.Logging.logFatal;
-;;;     }
+;;;     };
 ;;;     // make convenience url
 ;;;     //    javascript:kukit.showLog();
 ;;;     // instead of the need to say
@@ -236,7 +236,7 @@
 ;;;         this.logWarning = function(str) { console.log('WARNING: '+str); };
 ;;;         this.logError = function(str) { console.log('ERROR: '+str); };
 ;;;         this.logFatal = function(str) { console.log('FATAL: '+str); };
-;;;     }
+;;;     };
 ;;; }
 
 // Initialize the logger with the solution we've just detected
_______________________________________________
Kukit-checkins mailing list
[email protected]
http://codespeak.net/mailman/listinfo/kukit-checkins

Reply via email to