Make XHPASTLinter correct too little space before opening braces

Summary: Previously, we would not correct missing space before "{".

Test Plan: Ran unit tests.

Reviewers: btrahan, jungejason, kiyoto

Reviewed By: jungejason

CC: aran, jungejason

Differential Revision: https://secure.phabricator.com/D1313

src/lint/linter/xhpast/ArcanistXHPASTLinter.php

@@ -168,7 +168,24 @@ class ArcanistXHPASTLinter extends ArcanistLinter {
         continue;
       }
       list($before, $after) = $list->getSurroundingNonsemanticTokens();
-      if (count($before) == 1) {
+      if (!$before) {
+        $first = head($tokens);
+
+        // Only insert the space if we're after a closing parenthesis. If
+        // we're in a construct like "else{}", other rules will insert space
+        // after the 'else' correctly.
+        $prev = $first->getPrevToken();
+        if (!$prev || $prev->getValue() != ')') {
+          continue;
+        }
+
+        $this->raiseLintAtToken(
+          $first,
+          self::LINT_FORMATTING_CONVENTIONS,
+          'Put opening braces on the same line as control statements and '.
+          'declarations, with a single space before them.',
+          ' '.$first->getValue());
+      } else if (count($before) == 1) {
         $before = reset($before);
         if ($before->getValue() != ' ') {
           $this->raiseLintAtToken(

src/lint/linter/xhpast/__tests__/data/creative-brace-use.lint-test

@@ -25,6 +25,8 @@ try
 {}
 catch (Exception $x)
 {}
+
+function h(){}
 ~~~~~~~~~~
 warning:7:13
 warning:12:7
@@ -33,6 +35,7 @@ warning:18:10
 warning:21:11
 warning:24:4
 warning:26:21
+warning:29:13
 ~~~~~~~~~~
 <?php
 
@@ -54,3 +57,5 @@ switch (1) {}
 
 try {}
 catch (Exception $x) {}
+
+function h() {}