Merge pull request microsoft#4347 from Schmavery/master

mhegazy · mhegazy · commit 387da2fc1934 · 2015-08-18T09:51:27.000-07:00
Add `arguments` to completion list microsoft#4249
diff --git a/src/compiler/checker.ts b/src/compiler/checker.ts
@@ -13845,7 +13845,11 @@ namespace ts {
                             }
                             break;
                     }
-
+                    
+                    if (introducesArgumentsExoticObject(location)) {
+                        copySymbol(argumentsSymbol, meaning);
+                    }
+                    
                     memberFlags = location.flags;
                     location = location.parent;
                 }
diff --git a/src/compiler/utilities.ts b/src/compiler/utilities.ts
@@ -420,7 +420,10 @@ namespace ts {
     }
 
     export function getJsDocComments(node: Node, sourceFileOfNode: SourceFile) {
-        let commentRanges = (node.kind === SyntaxKind.Parameter || node.kind === SyntaxKind.TypeParameter) ?            concatenate(getTrailingCommentRanges(sourceFileOfNode.text, node.pos),                getLeadingCommentRanges(sourceFileOfNode.text, node.pos)) :            getLeadingCommentRangesOfNode(node, sourceFileOfNode);
+        let commentRanges = (node.kind === SyntaxKind.Parameter || node.kind === SyntaxKind.TypeParameter) ?
+            concatenate(getTrailingCommentRanges(sourceFileOfNode.text, node.pos),
+                getLeadingCommentRanges(sourceFileOfNode.text, node.pos)) :
+            getLeadingCommentRangesOfNode(node, sourceFileOfNode);
         return filter(commentRanges, isJsDocComment);
 
         function isJsDocComment(comment: CommentRange) {
@@ -638,6 +641,20 @@ namespace ts {
         return false;
     }
 
+    export function introducesArgumentsExoticObject(node: Node) {
+        switch (node.kind) {
+            case SyntaxKind.MethodDeclaration:
+            case SyntaxKind.MethodSignature:
+            case SyntaxKind.Constructor:
+            case SyntaxKind.GetAccessor:
+            case SyntaxKind.SetAccessor:
+            case SyntaxKind.FunctionDeclaration:
+            case SyntaxKind.FunctionExpression:
+                return true;
+        }
+        return false;
+    }
+
     export function isFunctionBlock(node: Node) {
         return node && node.kind === SyntaxKind.Block && isFunctionLike(node.parent);
     }
diff --git a/tests/cases/fourslash/completionInsideFunctionContainsArguments.ts b/tests/cases/fourslash/completionInsideFunctionContainsArguments.ts
@@ -0,0 +1,22 @@
+/// <reference path='fourslash.ts'/>
+
+////function testArguments() {/*1*/}
+/////*2*/
+////function testNestedArguments() {
+////  function nestedfunction(){/*3*/}
+////}
+////function f() {
+////    let g = () => /*4*/
+////}
+////let g = () => /*5*/
+
+goTo.marker('1');
+verify.completionListContains("arguments");
+goTo.marker('2');
+verify.not.completionListContains("arguments");
+goTo.marker('3');
+verify.completionListContains("arguments");
+goTo.marker('4');
+verify.completionListContains("arguments");
+goTo.marker('5');
+verify.not.completionListContains("arguments");

Original file line number	Diff line number	Diff line change
`@@ -13845,7 +13845,11 @@ namespace ts {`
`13845`	`13845`	`}`
`13846`	`13846`	`break;`
`13847`	`13847`	`}`
`13848`		`-`
	`13848`	`+`
	`13849`	`+ if (introducesArgumentsExoticObject(location)) {`
	`13850`	`+ copySymbol(argumentsSymbol, meaning);`
	`13851`	`+ }`
	`13852`	`+`
`13849`	`13853`	`memberFlags = location.flags;`
`13850`	`13854`	`location = location.parent;`
`13851`	`13855`	`}`