Skip to content

Add detect() method for the tab-size.js #520

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
drugan wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Add detect() method for the tab-size.js #520

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6954668
Add detect() method for the tab-size.js
May 17, 2017
4dbb02e
Improved logic for tab-size.js detect() and fixed a bug when running …
Jun 2, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Add detect() method for the tab-size.js
  • Loading branch information
drugan committed May 17, 2017
commit 6954668eda5698f9cc155bb1364234ce59ed7b33
33 changes: 33 additions & 0 deletions src/options/tab-size.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,38 @@ module.exports = {
ast.traverseByType('space', function(space) {
space.content = space.content.replace(/\t/, value);
});
},

/**
* Detects the value of this option in ast.
* @param {Node} ast
* @return {Array?} List of detected values
*/
detect(ast) {
var detected = [];

ast.traverseByType('block', block => {
var spaces = 0;
var tabs = 0;

block.forEach(blockContent => {
if (blockContent.is('space')) {
spaces = blockContent.content.replace(/\n/g, '');
tabs = spaces.match(/\t/g);
// It is very rare case when tabs are used in a file instead of
// whitespaces and it requires a lot of efforts to find the tab size
// in a particular environment, ... so we imply that tab size = 2.
tabs = tabs ? tabs.length * 2 : 0;
spaces = spaces.match(/( )/g);
spaces = spaces ? spaces.length + tabs : tabs;

} else if (spaces && blockContent.is('declaration')) {
detected.push(spaces);
spaces = 0;
}
});
});

return detected;
}
};