| Index: lib/src/rules/whitespace_around_ops.dart
|
| diff --git a/lib/src/rules/whitespace_around_ops.dart b/lib/src/rules/whitespace_around_ops.dart
|
| deleted file mode 100644
|
| index 38ae5ca7b203c316424c95856a1a56d03d774504..0000000000000000000000000000000000000000
|
| --- a/lib/src/rules/whitespace_around_ops.dart
|
| +++ /dev/null
|
| @@ -1,71 +0,0 @@
|
| -// Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file
|
| -// for details. All rights reserved. Use of this source code is governed by a
|
| -// BSD-style license that can be found in the LICENSE file.
|
| -
|
| -library linter.src.rules.whitespace_around_ops;
|
| -
|
| -import 'package:analyzer/dart/ast/ast.dart';
|
| -import 'package:analyzer/dart/ast/token.dart';
|
| -import 'package:analyzer/dart/ast/visitor.dart';
|
| -import 'package:linter/src/linter.dart';
|
| -
|
| -const desc = r'Use proper whitespace around operators.';
|
| -
|
| -const details = r'''
|
| -**DO** ensure that there are spaces around binary operators and before any
|
| -unary ones.
|
| -
|
| -Improper whitespace can create confusion, especially when applied to operators
|
| -where it's possible to get a binary operator when you mean a unary one. For
|
| -example, the mistyping of `5 /~ 10` when you mean `5 ~/ 10` is hidden by the
|
| -improper spacing. (Properly spaced, the mistake is more clear: `5 / ~10`.)
|
| -Whenever possible, use the formatter to cleanup whitespace. Otherwise, take
|
| -care to ensure that there are spaces around binary operators and before any
|
| -unary ones.
|
| -
|
| -
|
| -**BAD:**
|
| -```
|
| -print(5 /~ 10); //whoops
|
| -```
|
| -
|
| -**GOOD:**
|
| -```
|
| -print(5 / ~10); //aha!
|
| -```
|
| -''';
|
| -
|
| -class Visitor extends SimpleAstVisitor {
|
| - final LintRule rule;
|
| - Visitor(this.rule);
|
| -
|
| - @override
|
| - visitBinaryExpression(BinaryExpression node) {
|
| - if (!spaced(node.leftOperand.endToken, node.operator) ||
|
| - !spaced(node.operator, node.rightOperand.beginToken)) {
|
| - rule.reportLintForToken(node.operator);
|
| - }
|
| - }
|
| -
|
| - @override
|
| - visitPrefixExpression(PrefixExpression node) {
|
| - if (spaced(node.operator, node.operand.beginToken)) {
|
| - rule.reportLintForToken(node.operator);
|
| - }
|
| - }
|
| -
|
| - static bool spaced(Token first, Token second) =>
|
| - first != null && second != null && first.end != second.offset;
|
| -}
|
| -
|
| -class WhitespaceAroundOps extends LintRule {
|
| - WhitespaceAroundOps()
|
| - : super(
|
| - name: 'whitespace_around_ops',
|
| - description: desc,
|
| - details: details,
|
| - group: Group.style);
|
| -
|
| - @override
|
| - AstVisitor getVisitor() => new Visitor(this);
|
| -}
|
|
|