From ea3fddf53127a032c7412380ce0aa92b8310605e Mon Sep 17 00:00:00 2001
From: Jason Killian <jkillian@palantir.com>
Date: Thu, 10 Sep 2015 14:38:39 -0400
Subject: [PATCH] Don't check parameters in call signatures

---
 lib/tslint.d.ts                               | 1 +
 src/language/walker/syntaxWalker.ts           | 8 ++++++++
 src/rules/noShadowedVariableRule.ts           | 4 ++++
 test/files/rules/no-shadowed-variable.test.ts | 5 +++++
 4 files changed, 18 insertions(+)

diff --git a/lib/tslint.d.ts b/lib/tslint.d.ts
index a1c460059af..a4075eda50c 100644
--- a/lib/tslint.d.ts
+++ b/lib/tslint.d.ts
@@ -8,6 +8,7 @@ declare module Lint {
         protected visitBlock(node: ts.Block): void;
         protected visitBreakStatement(node: ts.BreakOrContinueStatement): void;
         protected visitCallExpression(node: ts.CallExpression): void;
+        protected visitCallSignature(node: ts.SignatureDeclaration): void;
         protected visitCaseClause(node: ts.CaseClause): void;
         protected visitClassDeclaration(node: ts.ClassDeclaration): void;
         protected visitCatchClause(node: ts.CatchClause): void;
diff --git a/src/language/walker/syntaxWalker.ts b/src/language/walker/syntaxWalker.ts
index 20fe515ce46..bfb51072ef1 100644
--- a/src/language/walker/syntaxWalker.ts
+++ b/src/language/walker/syntaxWalker.ts
@@ -48,6 +48,10 @@ module Lint {
             this.walkChildren(node);
         }
 
+        protected visitCallSignature(node: ts.SignatureDeclaration) {
+            this.walkChildren(node);
+        }
+
         protected visitCaseClause(node: ts.CaseClause) {
             this.walkChildren(node);
         }
@@ -310,6 +314,10 @@ module Lint {
                     this.visitCallExpression(<ts.CallExpression> node);
                     break;
 
+                case ts.SyntaxKind.CallSignature:
+                    this.visitCallSignature(<ts.SignatureDeclaration> node);
+                    break;
+
                 case ts.SyntaxKind.CaseClause:
                     this.visitCaseClause(<ts.CaseClause> node);
                     break;
diff --git a/src/rules/noShadowedVariableRule.ts b/src/rules/noShadowedVariableRule.ts
index 8fe02139a36..03fc1d6e28c 100644
--- a/src/rules/noShadowedVariableRule.ts
+++ b/src/rules/noShadowedVariableRule.ts
@@ -48,6 +48,10 @@ class NoShadowedVariableWalker extends Lint.BlockScopeAwareRuleWalker<ScopeInfo,
         this.visitBlock(node.block);
     }
 
+    public visitCallSignature(node: ts.SignatureDeclaration) {
+        // don't call super, we don't need to check parameter names in call signatures
+    }
+
     public visitFunctionType(node: ts.FunctionOrConstructorTypeNode) {
         // don't call super, we don't need to check names in function types
     }
diff --git a/test/files/rules/no-shadowed-variable.test.ts b/test/files/rules/no-shadowed-variable.test.ts
index 993cc939a09..126727bcb61 100644
--- a/test/files/rules/no-shadowed-variable.test.ts
+++ b/test/files/rules/no-shadowed-variable.test.ts
@@ -109,3 +109,8 @@ interface FalsePositive3 {
     specular: (pos: number) => number;
     pos: number;
 }
+
+interface FalsePositive4<T, TResult> {
+    (parameters: T, runSynchonous: boolean): TResult;
+    (parameters: T, callback: (error: Error, result: TResult) => void): void;
+}