Chore: remove unused code in max-lines-per-function

[yeonjuan]

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ x ] Other, please explain:

What changes did you make? (Give an overview)

remove unused code in max-lines-per-function.

eslint/lib/rules/max-lines-per-function.js

Lines 57 to 59 in 8108f49

	if (!comments) {
	return map;
	}

Because the return value from sourceCode.getAllComments() is always truthy, checking falsy is useless.

• in other rules, they didn't check whether the return value is falsy or not.
  

  eslint/lib/rules/line-comment-position.js

  Lines 89 to 93 in 219aecb

  	const comments = sourceCode.getAllComments();
  	comments.filter(token => token.type === "Line").forEach(node => {
  	if (applyDefaultIgnorePatterns && (defaultIgnoreRegExp.test(node.value) || fallThroughRegExp.test(node.value))) {
  	return;

  
  

  eslint/lib/rules/capitalized-comments.js

  Lines 294 to 296 in 8108f49

  	const comments = sourceCode.getAllComments();
  	comments.filter(token => token.type !== "Shebang").forEach(processComment);

Is there anything you'd like reviewers to focus on?

[kaicataldo]

Thanks for contributing!