kotovalexarian-likes-github/jashkenas--coffeescript
1
0
Fork 0
mirror of https://github.com/jashkenas/coffeescript.git synced 2022-11-09 12:23:24 -05:00
Code Releases Activity

fixed a bug where postfix ? was incorrectly unwrapping its operand

Browse source
This commit is contained in:
satyr 2010-11-09 16:27:21 +09:00
parent 188ad03c17
commit 3ae818860b
3 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/nodes.js
View file

@ -1594,7 +1594,7 @@
Existence.prototype.invert = NEGATE;
Existence.prototype.compileNode = function(o) {
var code, sym;
code = this.expression.compile(o);
code = this.expression.compile(o, LEVEL_OP);
code = IDENTIFIER.test(code) && !o.scope.check(code) ? this.negated ? "typeof " + code + " == \"undefined\" || " + code + " === null" : "typeof " + code + " != \"undefined\" && " + code + " !== null" : (sym = this.negated ? '==' : '!=', "" + code + " " + sym + " null");
if (o.level <= LEVEL_COND) {
return code;

2
src/nodes.coffee
View file

@ -1216,7 +1216,7 @@ exports.Existence = class Existence extends Base
constructor: (@expression) ->
compileNode: (o) ->
code = @expression.compile o
code = @expression.compile o, LEVEL_OP
code = if IDENTIFIER.test(code) and not o.scope.check code
if @negated
"typeof #{code} == \"undefined\" || #{code} === null"

3
test/test_existence.coffee
View file

@ -160,3 +160,6 @@ eq 1, a?.b.c + 1
eq 1, a?.b.c += 1
eq 2, ++a?.b.c
eq yes, delete a?.b.c
eq (1 or 0)?, true, 'postfix `?` should unwrap correctly'