Tighten no ++/-- rule

sandersn · sandersn · commit 189357db1aa0 · 2015-12-22T15:37:24.000-08:00
diff --git a/scripts/tslint/noIncrementDecrementRule.ts b/scripts/tslint/noIncrementDecrementRule.ts
@@ -28,7 +28,8 @@ class IncrementDecrementWalker extends Lint.RuleWalker {
     }
 
     visitIncrementDecrement(node: ts.UnaryExpression) {
-        if (node.parent && (node.parent.kind === ts.SyntaxKind.ExpressionStatement || node.parent.kind === ts.SyntaxKind.ForStatement || node.parent.kind === ts.SyntaxKind.ElementAccessExpression)) {
+        if (node.parent && (node.parent.kind === ts.SyntaxKind.ExpressionStatement ||
+                            node.parent.kind === ts.SyntaxKind.ForStatement)) {
             return;
         }
         this.addFailure(this.createFailure(node.getStart(), node.getWidth(), Rule.POSTFIX_FAILURE_STRING));

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,8 @@ class IncrementDecrementWalker extends Lint.RuleWalker {`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`visitIncrementDecrement(node: ts.UnaryExpression) {`
`31`		`- if (node.parent && (node.parent.kind === ts.SyntaxKind.ExpressionStatement \|\| node.parent.kind === ts.SyntaxKind.ForStatement \|\| node.parent.kind === ts.SyntaxKind.ElementAccessExpression)) {`
	`31`	`+ if (node.parent && (node.parent.kind === ts.SyntaxKind.ExpressionStatement \|\|`
	`32`	`+ node.parent.kind === ts.SyntaxKind.ForStatement)) {`
`32`	`33`	`return;`
`33`	`34`	`}`
`34`	`35`	`this.addFailure(this.createFailure(node.getStart(), node.getWidth(), Rule.POSTFIX_FAILURE_STRING));`