x550j / trimpath

Automatically exported from code.google.com/p/trimpath
0 stars 0 forks source link

This looks like a probably error? #2

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
In your definition of the keywords, the NEQ keyword has EQ as its string in
the definition (line 365), that looks like a copy/paste error to me, but
what do I know?

        var keywords = {
            SELECT_ALL      : function() { return new
NodeType.select(arguments); },
            SELECT_DISTINCT : TODO,
            ALL   : USAGE, // We use ALL in different syntax, like SELECT_ALL.
            FROM  : function() { return new NodeType.from(arguments); },
            WHERE : function() { return new NodeType.where(arguments); },
            AND   : function() { return new NodeType.expression(arguments,
"AND",  0, null, 1, null, "&&"); },
            OR    : function() { return new NodeType.expression(arguments,
"OR",   0, null, 1, null, "||"); },
            NOT   : function() { return new NodeType.expression(arguments,
"NOT", -1, null, 1, 1, "!"); },
            EQ    : function() { return new NodeType.expression(arguments,
"EQ",   0, "=",  2, 2, "=="); },
            NEQ   : function() { return new NodeType.expression(arguments,
"EQ",   0, "!=", 2, 2); },
            LT    : function() { return new NodeType.expression(arguments,
"LT",   0, "<",  2, 2); },
            GT    : function() { return new NodeType.expression(arguments,
"GT",   0, ">",  2, 2); },
            LTE   : function() { return new NodeType.expression(arguments,
"LTE",  0, "<=", 2, 2); },
            GTE   : function() { return new NodeType.expression(arguments,
"GTE",  0, ">=", 2, 2); },

Original issue reported on code.google.com by carlemueller on 27 Jun 2007 at 5:55

GoogleCodeExporter commented 8 years ago
Thanks.  Just checked in the fix in.  Probably out in junction release 1.1.4

Original comment by steve....@gmail.com on 16 Jul 2007 at 5:15

GoogleCodeExporter commented 8 years ago
You can get the fixed version as part of the trimpath-junction-VER.tar.gz 
package at
http://code.google.com/p/trimpath/downloads/list

Original comment by steve....@gmail.com on 6 Aug 2007 at 7:04