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

Add disallowEmptyBlocks rule#122

Merged
mdevils merged 2 commits intojscs-dev:masterfrom
mikesherov:noempty
Dec 30, 2013
Merged

Add disallowEmptyBlocks rule#122
mdevils merged 2 commits intojscs-dev:masterfrom
mikesherov:noempty

Conversation

@mikesherov
Copy link
Copy Markdown
Contributor

No description provided.

@mikesherov mikesherov restored the noempty branch December 30, 2013 01:19
@mdevils
Copy link
Copy Markdown
Member

mdevils commented Dec 30, 2013

Why catch is an exception?

@mgol
Copy link
Copy Markdown

mgol commented Dec 30, 2013

@mdevils Because it's impossible otherwise to silence errors. An 'if', 'else' etc. with an empty body can always be removed but the 'try' block is impossible without 'catch' or 'finally'.

@mikesherov
Copy link
Copy Markdown
Contributor Author

@mdevils exactly what @mzgol said.

@mdevils
Copy link
Copy Markdown
Member

mdevils commented Dec 30, 2013

Thank you for your submission!

mdevils added a commit that referenced this pull request Dec 30, 2013
Add disallowEmptyBlocks rule
@mdevils mdevils merged commit d94786a into jscs-dev:master Dec 30, 2013
@mikesherov mikesherov deleted the noempty branch December 30, 2013 19:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants