Enable "no-unnecessary-callback-wrapper"

src/rules/typedefRule.ts

@@ -221,37 +221,13 @@ class TypedefWalker extends Lint.RuleWalker {
     private checkTypeAnnotation(option: string,
                                 location: number,
                                 typeAnnotation: ts.TypeNode | undefined,
-                                name?: ts.Node) {
-        if (this.hasOption(option) && typeAnnotation == null) {
-            this.addFailureAt(location, 1, "expected " + option + getName(name, ": '", "'") + " to have a typedef");
+                                name?: ts.PropertyName | ts.BindingName) {
+        if (this.hasOption(option) && typeAnnotation === undefined) {
+            this.addFailureAt(location, 1, `expected ${option}${name === undefined ? "" : `: '${name.getText()}'`} to have a typedef`);
         }
     }
 }
 
-function getName(name?: ts.Node, prefix = "", suffix = ""): string {
-    let ns = "";
-
-    if (name != null) {
-        switch (name.kind) {
-            case ts.SyntaxKind.Identifier:
-                ns = (name as ts.Identifier).text;
-                break;
-            case ts.SyntaxKind.BindingElement:
-                ns = getName((name as ts.BindingElement).name);
-                break;
-            case ts.SyntaxKind.ArrayBindingPattern:
-                ns = `[ ${(name as ts.ArrayBindingPattern).elements.map( (n) => getName(n) ).join(", ")} ]`;
-                break;
-            case ts.SyntaxKind.ObjectBindingPattern:
-                ns = `{ ${(name as ts.ObjectBindingPattern).elements.map( (n) => getName(n) ).join(", ")} }`;
-                break;
-            default:
-                break;
-        }
-    }
-    return ns === "" ? "" : `${prefix}${ns}${suffix}`;
-}
-
 function isTypedPropertyDeclaration(node: ts.Node): boolean {
     return utils.isPropertyDeclaration(node) && node.type != null;
 }

tslint.json

@@ -52,7 +52,6 @@
     "no-null-keyword": false,
     "no-require-imports": false,
     "no-unbound-method": false,
-    "no-unnecessary-callback-wrapper": false,
     "no-unnecessary-type-assertion": false,
     "no-unnecessary-qualifier": false,
     "no-use-before-declare": false,