From e62c42b8f45a1df9cfac11eea931f7e03367ea7e Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Tue, 26 Apr 2016 22:14:28 -0700
Subject: [PATCH] tools: remove default parameters from lint rule
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The CI server uses system Node.js for linting, which is currently v5.x.
So default parameters are not supported there. This change removes the
default parameters.

PR-URL: https://github.com/nodejs/node/pull/6411
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 tools/eslint-rules/align-multiline-assignment.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/eslint-rules/align-multiline-assignment.js b/tools/eslint-rules/align-multiline-assignment.js
index 69f1f33d4d184f..80896b5e96296d 100644
--- a/tools/eslint-rules/align-multiline-assignment.js
+++ b/tools/eslint-rules/align-multiline-assignment.js
@@ -7,7 +7,7 @@
 //------------------------------------------------------------------------------
 // Rule Definition
 //------------------------------------------------------------------------------
-function getBinaryExpressionStarts(binaryExpression, starts = []) {
+function getBinaryExpressionStarts(binaryExpression, starts) {
   function getStartsFromOneSide(side, starts) {
     starts.push(side.loc.start);
     if (side.type === 'BinaryExpression') {
@@ -29,7 +29,7 @@ function checkExpressionAlignment(expression) {
 
   switch (expression.type) {
     case 'BinaryExpression':
-      var starts = getBinaryExpressionStarts(expression);
+      var starts = getBinaryExpressionStarts(expression, []);
       var startLine = starts[0].line;
       const startColumn = starts[0].column;
       starts.forEach((loc) => {