D14499.id35074.diff
No OneTemporary
Actions

Size

876 B

Referenced Files

None

Subscribers

None

D14499.id35074.diff
View Options

	diff --git a/src/lint/linter/xhpast/rules/ArcanistBraceFormattingXHPASTLinterRule.php b/src/lint/linter/xhpast/rules/ArcanistBraceFormattingXHPASTLinterRule.php
	--- a/src/lint/linter/xhpast/rules/ArcanistBraceFormattingXHPASTLinterRule.php
	+++ b/src/lint/linter/xhpast/rules/ArcanistBraceFormattingXHPASTLinterRule.php
	@@ -61,10 +61,17 @@
	}

	$type = $parent->getTypeName();
	- if ($type != 'n_STATEMENT_LIST' && $type != 'n_DECLARE') {
	- $this->raiseLintAtNode(
	- $node,
	- pht('Use braces to surround a statement block.'));
	+ switch ($type) {
	+ case 'n_DECLARE':
	+ case 'n_NAMESPACE':
	+ case 'n_STATEMENT_LIST':
	+ break;
	+
	+ default:
	+ $this->raiseLintAtNode(
	+ $node,
	+ pht('Use braces to surround a statement block.'));
	+ break;
	}
	}