CS: remove unneeded parentheses around control statements

keradus · keradus · commit 45d250d25f76 · 2015-12-01T12:58:24.000+01:00
diff --git a/src/Symfony/Bridge/Doctrine/Form/ChoiceList/ORMQueryBuilderLoader.php b/src/Symfony/Bridge/Doctrine/Form/ChoiceList/ORMQueryBuilderLoader.php
@@ -76,7 +76,7 @@ public function getEntities()
      */
     public function getEntitiesByIds($identifier, array $values)
     {
-        $qb = clone ($this->queryBuilder);
+        $qb = clone $this->queryBuilder;
         $alias = current($qb->getRootAliases());
         $parameter = 'ORMQueryBuilderLoader_getEntitiesByIds_'.$identifier;
         $where = $qb->expr()->in($alias.'.'.$identifier, ':'.$parameter);
diff --git a/src/Symfony/Component/Filesystem/Filesystem.php b/src/Symfony/Component/Filesystem/Filesystem.php
@@ -428,13 +428,13 @@ public function mirror($originDir, $targetDir, \Traversable $iterator = null, $o
      */
     public function isAbsolutePath($file)
     {
-        return (strspn($file, '/\\', 0, 1)
+        return strspn($file, '/\\', 0, 1)
             || (strlen($file) > 3 && ctype_alpha($file[0])
                 && substr($file, 1, 1) === ':'
                 && (strspn($file, '/\\', 2, 1))
             )
             || null !== parse_url($file, PHP_URL_SCHEME)
-        );
+        ;
     }
 
     /**
diff --git a/src/Symfony/Component/Finder/Iterator/SortableIterator.php b/src/Symfony/Component/Finder/Iterator/SortableIterator.php
@@ -55,15 +55,15 @@ public function __construct(\Traversable $iterator, $sort)
             };
         } elseif (self::SORT_BY_ACCESSED_TIME === $sort) {
             $this->sort = function ($a, $b) {
-                return ($a->getATime() - $b->getATime());
+                return $a->getATime() - $b->getATime();
             };
         } elseif (self::SORT_BY_CHANGED_TIME === $sort) {
             $this->sort = function ($a, $b) {
-                return ($a->getCTime() - $b->getCTime());
+                return $a->getCTime() - $b->getCTime();
             };
         } elseif (self::SORT_BY_MODIFIED_TIME === $sort) {
             $this->sort = function ($a, $b) {
-                return ($a->getMTime() - $b->getMTime());
+                return $a->getMTime() - $b->getMTime();
             };
         } elseif (is_callable($sort)) {
             $this->sort = $sort;
diff --git a/src/Symfony/Component/HttpFoundation/Session/Storage/Proxy/AbstractProxy.php b/src/Symfony/Component/HttpFoundation/Session/Storage/Proxy/AbstractProxy.php
@@ -52,7 +52,7 @@ public function getSaveHandlerName()
      */
     public function isSessionHandlerInterface()
     {
-        return ($this instanceof \SessionHandlerInterface);
+        return $this instanceof \SessionHandlerInterface;
     }
 
     /**
diff --git a/src/Symfony/Component/Intl/DateFormatter/DateFormat/FullTransformer.php b/src/Symfony/Component/Intl/DateFormatter/DateFormat/FullTransformer.php
@@ -214,7 +214,7 @@ public function getReverseMatchingRegExp($pattern)
      */
     public function isQuoteMatch($quoteMatch)
     {
-        return ("'" === $quoteMatch[0]);
+        return "'" === $quoteMatch[0];
     }
 
     /**
diff --git a/src/Symfony/Component/Serializer/Normalizer/GetSetMethodNormalizer.php b/src/Symfony/Component/Serializer/Normalizer/GetSetMethodNormalizer.php
@@ -242,10 +242,10 @@ private function supports($class)
      */
     private function isGetMethod(\ReflectionMethod $method)
     {
-        return (
+        return
             0 === strpos($method->name, 'get') &&
             3 < strlen($method->name) &&
             0 === $method->getNumberOfRequiredParameters()
-        );
+        ;
     }
 }
diff --git a/src/Symfony/Component/Yaml/Parser.php b/src/Symfony/Component/Yaml/Parser.php
@@ -670,6 +670,6 @@ private function isNextLineUnIndentedCollection()
      */
     private function isStringUnIndentedCollectionItem()
     {
-        return (0 === strpos($this->currentLine, '- '));
+        return 0 === strpos($this->currentLine, '- ');
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ public function getEntities()`
`76`	`76`	`*/`
`77`	`77`	`public function getEntitiesByIds($identifier, array $values)`
`78`	`78`	`{`
`79`		`- $qb = clone ($this->queryBuilder);`
	`79`	`+ $qb = clone $this->queryBuilder;`
`80`	`80`	`$alias = current($qb->getRootAliases());`
`81`	`81`	`$parameter = 'ORMQueryBuilderLoader_getEntitiesByIds_'.$identifier;`
`82`	`82`	`$where = $qb->expr()->in($alias.'.'.$identifier, ':'.$parameter);`
Original file line number	Diff line number	Diff line change
`@@ -428,13 +428,13 @@ public function mirror($originDir, $targetDir, \Traversable $iterator = null, $o`
`428`	`428`	`*/`
`429`	`429`	`public function isAbsolutePath($file)`
`430`	`430`	`{`
`431`		`- return (strspn($file, '/\\', 0, 1)`
	`431`	`+ return strspn($file, '/\\', 0, 1)`
`432`	`432`	`\|\| (strlen($file) > 3 && ctype_alpha($file[0])`
`433`	`433`	`&& substr($file, 1, 1) === ':'`
`434`	`434`	`&& (strspn($file, '/\\', 2, 1))`
`435`	`435`	`)`
`436`	`436`	`\|\| null !== parse_url($file, PHP_URL_SCHEME)`
`437`		`- );`
	`437`	`+ ;`
`438`	`438`	`}`
`439`	`439`
`440`	`440`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public function getSaveHandlerName()`
`52`	`52`	`*/`
`53`	`53`	`public function isSessionHandlerInterface()`
`54`	`54`	`{`
`55`		`- return ($this instanceof \SessionHandlerInterface);`
	`55`	`+ return $this instanceof \SessionHandlerInterface;`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ public function getReverseMatchingRegExp($pattern)`
`214`	`214`	`*/`
`215`	`215`	`public function isQuoteMatch($quoteMatch)`
`216`	`216`	`{`
`217`		`- return ("'" === $quoteMatch[0]);`
	`217`	`+ return "'" === $quoteMatch[0];`
`218`	`218`	`}`
`219`	`219`
`220`	`220`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -242,10 +242,10 @@ private function supports($class)`
`242`	`242`	`*/`
`243`	`243`	`private function isGetMethod(\ReflectionMethod $method)`
`244`	`244`	`{`
`245`		`- return (`
	`245`	`+ return`
`246`	`246`	`0 === strpos($method->name, 'get') &&`
`247`	`247`	`3 < strlen($method->name) &&`
`248`	`248`	`0 === $method->getNumberOfRequiredParameters()`
`249`		`- );`
	`249`	`+ ;`
`250`	`250`	`}`
`251`	`251`	`}`
Original file line number	Diff line number	Diff line change
`@@ -670,6 +670,6 @@ private function isNextLineUnIndentedCollection()`
`670`	`670`	`*/`
`671`	`671`	`private function isStringUnIndentedCollectionItem()`
`672`	`672`	`{`
`673`		`- return (0 === strpos($this->currentLine, '- '));`
	`673`	`+ return 0 === strpos($this->currentLine, '- ');`
`674`	`674`	`}`
`675`	`675`	`}`