Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

new new f a #1416

Closed
satyr opened this issue Jun 3, 2011 · 2 comments · Fixed by #1450
Closed

new new f a #1416

satyr opened this issue Jun 3, 2011 · 2 comments · Fixed by #1450
Assignees
@michaelficarra
Labels
bug

Comments

@satyr
Copy link
Collaborator

satyr commented Jun 3, 2011 

$ coffee -bpe 'new new f a'
new f(a);

$ coffee -e 'console.log new new Function "++this.$"'
[Function: anonymous]

$ node -e 'new new Function("++this.$")'
{ '$': NaN }
thejh added a commit to thejh/coffee-script that referenced this issue Jun 20, 2011
@thejh
test for jashkenas#1416: don't omit one 'new' when compiling 'new new'
11f2cd4
thejh added a commit to thejh/coffee-script that referenced this issue Jun 20, 2011
@thejh
fixes jashkenas#1416: don't omit one 'new' when compiling 'new new'
f0c22f3
@thejh
Copy link
Contributor

thejh commented Jun 20, 2011

Pull request: #1450

thejh added a commit to thejh/coffee-script that referenced this issue Jun 21, 2011
@thejh
added another test for jashkenas#1416
8b2884e
thejh added a commit to thejh/coffee-script that referenced this issue Jun 21, 2011
@thejh
second part of fixing jashkenas#1416
73af3b1
@ghost ghost assigned michaelficarra Jun 24, 2011
michaelficarra added a commit that referenced this issue Jun 30, 2011
@michaelficarra
improved tests for #1416, fixed accidental scope leak in test for #1420
7b5f012
@michaelficarra
Copy link
Collaborator

@thejh's fix has been merged, closing

michaelficarra pushed a commit that referenced this issue Jul 7, 2011
@thejh @michaelficarra
test for #1416: don't omit one 'new' when compiling 'new new'
4c70ea5
michaelficarra pushed a commit that referenced this issue Jul 7, 2011
@thejh @michaelficarra
added another test for #1416
9d3510a
michaelficarra pushed a commit that referenced this issue Jul 7, 2011
@thejh @michaelficarra
second part of fixing #1416
4af47f0
michaelficarra added a commit that referenced this issue Jul 7, 2011
@michaelficarra
improved tests for #1416, fixed accidental scope leak in test for #1420
594ead0
breckinloggins pushed a commit to breckinloggins/coffee-script that referenced this issue Jul 9, 2011
@thejh @michaelficarra
fixes jashkenas#1416: don't omit one 'new' when compiling 'new new'
18f6ad9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michaelficarra michaelficarra

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bugfix for #1416 thejh/coffee-script
3 participants
@satyr @michaelficarra @thejh