Skip to content
This repository was archived by the owner on Mar 23, 2024. It is now read-only.

Commit b33d03e

Browse files
hzoomikesherov
authored andcommitted
(disallow|require)SpacesInForStatements: use tokenAssert
1 parent ca2ea32 commit b33d03e

2 files changed

Lines changed: 10 additions & 18 deletions

File tree

lib/rules/disallow-spaces-in-for-statement.js

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -58,24 +58,19 @@ module.exports.prototype = {
5858

5959
check: function(file, errors) {
6060
file.iterateNodesByType('ForStatement', function(node) {
61-
var prevToken;
6261
if (node.test) {
63-
var testToken = file.getTokenByRangeStart(node.test.range[0]);
64-
prevToken = file.getPrevToken(testToken);
65-
errors.assert.whitespaceBetween({
66-
token: prevToken,
62+
var testToken = file.getFirstNodeToken(node.test);
63+
errors.assert.noWhitespaceBetween({
64+
token: file.getPrevToken(testToken),
6765
nextToken: testToken,
68-
spaces: 0,
6966
message: 'Space found after semicolon'
7067
});
7168
}
7269
if (node.update) {
73-
var updateToken = file.getTokenByRangeStart(node.update.range[0]);
74-
prevToken = file.getPrevToken(updateToken);
75-
errors.assert.whitespaceBetween({
76-
token: prevToken,
70+
var updateToken = file.getFirstNodeToken(node.update);
71+
errors.assert.noWhitespaceBetween({
72+
token: file.getPrevToken(updateToken),
7773
nextToken: updateToken,
78-
spaces: 0,
7974
message: 'Space found after semicolon'
8075
});
8176
}

lib/rules/require-spaces-in-for-statement.js

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -58,22 +58,19 @@ module.exports.prototype = {
5858

5959
check: function(file, errors) {
6060
file.iterateNodesByType('ForStatement', function(node) {
61-
var prevToken;
6261
if (node.test) {
63-
var testToken = file.getTokenByRangeStart(node.test.range[0]);
64-
prevToken = file.getPrevToken(testToken);
62+
var testToken = file.getFirstNodeToken(node.test);
6563
errors.assert.whitespaceBetween({
66-
token: prevToken,
64+
token: file.getPrevToken(testToken),
6765
nextToken: testToken,
6866
spaces: 1,
6967
message: 'One space required after semicolon'
7068
});
7169
}
7270
if (node.update) {
73-
var updateToken = file.getTokenByRangeStart(node.update.range[0]);
74-
prevToken = file.getPrevToken(updateToken);
71+
var updateToken = file.getFirstNodeToken(node.update);
7572
errors.assert.whitespaceBetween({
76-
token: prevToken,
73+
token: file.getPrevToken(updateToken),
7774
nextToken: updateToken,
7875
spaces: 1,
7976
message: 'One space required after semicolon'

0 commit comments

Comments
 (0)