Fix(flow): Allow unnamed parameters (#674)

This allows `type T = number => string`. Previously we were only
handling `type T = (x: number) => string` correctly.

Followup to #672 that:
 - Adds a test fixture
 - Fixes lint errors

Author: arv

lib/flow_doctrine.js

@@ -100,11 +100,11 @@ function flowDoctrine(type/*: Object */)/*: DoctrineType*/ {
       params: type.params.map(param => {
         let name = '';
         if (param.name && param.name.name) {
-            name = param.name.name;
+          name = param.name.name;
         }
         return {
           type: 'ParameterType',
-          name: name,
+          name,
           expression: flowDoctrine(param.typeAnnotation)
         };
       }),

lib/output/util/format_type.js

@@ -119,7 +119,7 @@ function formatType(getHref/*: Function*/, node/*: ?Object */) {
     return [link(node.name, getHref)];
   case Syntax.ParameterType:
     if (node.name) {
-        result.push(t(node.name + ': '));
+      result.push(t(node.name + ': '));
     }
     return result.concat(formatType(getHref, node.expression));
 

test/fixture/flow-unnamed-params.input.js

@@ -0,0 +1,15 @@
+// @flow
+
+'use strict';
+
+/** x */
+let x: T => string;
+
+/** x2 */
+let x2: (a: T) => string;
+
+/** T */
+type T = string[] => {num: number};
+
+/** T2 */
+type T2 = (a: string[]) => {num: number};