checker: minor style tweaks

Author: getify

LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2018 Kyle Simpson <[email protected]>
+Copyright (c) 2019 Kyle Simpson <[email protected]>
 
 Permission is hereby granted, free of charge, to any person
 obtaining a copy of this software and associated documentation

lib/checker.js

@@ -798,7 +798,7 @@ function registerFuncSignatureParams(path) {
 	var funcNode = path.node;
 	var funcSignature = typeSignatures.get(funcNode);
 
-	for (let [paramIdx,param] of funcNode.params.entries()) {
+	for (let [paramIdx,param,] of funcNode.params.entries()) {
 		let paramType = nodeTypes.get(param);
 		if (paramType) {
 			// don't save a ...rest element into the signature,
@@ -994,7 +994,7 @@ function handleBinarySelection(exprNode,leftNode,rightNode) {
 }
 
 function handleBinaryPlus(exprNode) {
-	var [leftType,rightType] = binaryExpressionTypes(exprNode);
+	var [leftType,rightType,] = binaryExpressionTypes(exprNode);
 	var leftTypeID = getTypeID(leftType);
 	var rightTypeID = getTypeID(rightType);
 
@@ -1064,7 +1064,7 @@ function handleUnaryNumeric(op,exprNode) {
 }
 
 function handleBinaryNumeric(op,exprNode) {
-	var [leftType,rightType] = binaryExpressionTypes(exprNode);
+	var [leftType,rightType,] = binaryExpressionTypes(exprNode);
 
 	var leftTypeID = getTypeID(leftType);
 	var rightTypeID = getTypeID(rightType);
@@ -1170,7 +1170,7 @@ function handleBinaryNumeric(op,exprNode) {
 }
 
 function handleBinaryRelativeComparison(op,exprNode) {
-	var [leftType,rightType] = binaryExpressionTypes(exprNode);
+	var [leftType,rightType,] = binaryExpressionTypes(exprNode);
 	var leftTypeID = getTypeID(leftType);
 	var rightTypeID = getTypeID(rightType);
 	var validIDs = [ "string", "number", "finite", "int", "bint", ];
@@ -1228,7 +1228,7 @@ function handleBinaryRelativeComparison(op,exprNode) {
 }
 
 function handleEquality(op,exprNode) {
-	var [leftType,rightType] = binaryExpressionTypes(exprNode);
+	var [leftType,rightType,] = binaryExpressionTypes(exprNode);
 	var leftTypeID = getTypeID(leftType);
 	var rightTypeID = getTypeID(rightType);
 
@@ -1412,15 +1412,15 @@ function handleAssignmentExpressionType(scope,exprNode,targetNode,sourceNode,isA
 		if (exprNode) {
 			let sourceType = nodeTypes.get(sourceNode);
 			if (sourceType) {
-				markNodeType(exprNode,{ ...sourceType });
+				markNodeType(exprNode,{ ...sourceType, });
 			}
 			else {
 				markNodeType(exprNode,{ inferred: "array", });
 			}
 		}
 
 		if (T.isArrayExpression(sourceNode)) {
-			for (let [idx,targetElem] of targetNode.elements.entries()) {
+			for (let [idx,targetElem,] of targetNode.elements.entries()) {
 				// target is identifier with a default = value assignment?
 				if (T.isAssignmentPattern(targetElem)) {
 					targetElem = targetElem.left;
@@ -1437,15 +1437,15 @@ function handleAssignmentExpressionType(scope,exprNode,targetNode,sourceNode,isA
 		if (exprNode) {
 			let sourceType = nodeTypes.get(sourceNode);
 			if (sourceType) {
-				markNodeType(exprNode,{ ...sourceType });
+				markNodeType(exprNode,{ ...sourceType, });
 			}
 			else {
 				markNodeType(exprNode,{ inferred: "object", });
 			}
 		}
 
 		if (T.isObjectExpression(sourceNode)) {
-			for (let [idx,targetProp] of targetNode.properties.entries()) {
+			for (let [idx,targetProp,] of targetNode.properties.entries()) {
 				let targetPropName = targetProp.key.name;
 				targetProp = targetProp.value;
 
@@ -1608,7 +1608,7 @@ function handleCallExpression(scope,callExprNode,isSelfRecursivePTC = false) {
 		let funcNodePath = signatureNodePaths.get(funcSignature);
 		let funcNodeScope = funcNodePath ? funcNodePath.scope : undefined;
 		let funcNode = funcNodePath ? funcNodePath.node : undefined;
-		for (let [argIdx,argType] of callExpressionArgumentTypes.entries()) {
+		for (let [argIdx,argType,] of callExpressionArgumentTypes.entries()) {
 			// did we hit a spread argument?
 			if (T.isSpreadElement(callExprNode.arguments[argIdx])) {
 				// stop comparing arguments/params because of spread
@@ -1812,7 +1812,7 @@ function signaturesMatch(signature1,signature2) {
 			return false;
 		}
 
-		for (let [idx,paramType1] of signature1.params.entries()) {
+		for (let [idx,paramType1,] of signature1.params.entries()) {
 			let paramType2 = signature2.params[idx];
 
 			if (getTypeID(paramType1) != getTypeID(paramType2)) {
@@ -1862,16 +1862,16 @@ function isAssignmentAllowed(sourceType,targetType) {
 		number: [ "number", "int", "finite", ],
 		finite: [ "finite", "int", ],
 		bint: [ "bint", "int", ],
-		string: [ "string", ]
-	}
+		string: [ "string", ],
+	};
 
-	let sourceTypeID = getTypeID(sourceType)
-	let targetTypeID = getTypeID(targetType)
+	let sourceTypeID = getTypeID(sourceType);
+	let targetTypeID = getTypeID(targetType);
 
 	return (
 		matches[targetTypeID] &&
 		matches[targetTypeID].includes(sourceTypeID)
-	)
+	);
 }
 
 function isNumberOrSubtype(type) {
@@ -1952,7 +1952,7 @@ function check(code, options = {}) {
 			...prevMsgs,
 			...(
 				outputMessages.length > 0 ?
-					[{ type: "info", text: `(pass ${passCount}) ------------------------`, }] :
+					[{ type: "info", text: `(pass ${passCount}) ------------------------`, },] :
 					[]
 			),
 			...outputMessages,