goaop / framework

src/Core/GoAspectContainer.php

Unused Use Statements -1 removed lines

@@ -16,7 +16,6 @@
 use Go\Aop\Pointcut\PointcutLexer;
 use Go\Aop\Pointcut\PointcutGrammar;
 use Go\Aop\Pointcut\PointcutParser;
-use Go\Instrument\RawAnnotationReader;
 use Go\Instrument\ClassLoading\CachePathManager;
 use Doctrine\Common\Annotations\AnnotationReader;
 

Spacing +9 added lines, -9 removed lines

@@ -45,7 +45,7 @@ 
     public function __construct()
     {
         // Register all services in the container
-        $this->share('aspect.loader', function (Container $container) {
+        $this->share('aspect.loader', function(Container $container) {
             $aspectLoader = new AspectLoader(
                 $container,
                 $container->get('aspect.annotation.reader')
@@ -60,7 +60,7 @@ 
             return $aspectLoader;
         });
 
-        $this->share('aspect.cached.loader', function (Container $container) {
+        $this->share('aspect.cached.loader', function(Container $container) {
             $cachedAspectLoader = new CachedAspectLoader(
                 $container,
                 'aspect.loader',
@@ -70,25 +70,25 @@ 
             return $cachedAspectLoader;
         });
 
-        $this->share('aspect.advisor.accessor', function (Container $container) {
+        $this->share('aspect.advisor.accessor', function(Container $container) {
             return new LazyAdvisorAccessor(
                 $container,
                 $container->get('aspect.cached.loader')
             );
         });
 
-        $this->share('aspect.advice_matcher', function (Container $container) {
+        $this->share('aspect.advice_matcher', function(Container $container) {
             return new AdviceMatcher(
                 $container->get('aspect.loader'),
                 $container->get('kernel.interceptFunctions')
             );
         });
 
-        $this->share('aspect.annotation.reader', function (Container $container) {
+        $this->share('aspect.annotation.reader', function(Container $container) {
             $options = $container->get('kernel.options');
             $reader  = new AnnotationReader();
             if (!empty($options['cacheDir'])) {
-                $reader  = new FileCacheReader(
+                $reader = new FileCacheReader(
                     $reader,
                     $options['cacheDir'] . DIRECTORY_SEPARATOR . '_annotations' . DIRECTORY_SEPARATOR,
                     $options['debug']
@@ -97,15 +97,15 @@ 
 
             return $reader;
         });
-        $this->share('aspect.cache.path.manager', function (Container $container) {
+        $this->share('aspect.cache.path.manager', function(Container $container) {
             return new CachePathManager($container->get('kernel'));
         });
 
         // Pointcut services
-        $this->share('aspect.pointcut.lexer', function () {
+        $this->share('aspect.pointcut.lexer', function() {
             return new PointcutLexer();
         });
-        $this->share('aspect.pointcut.parser', function (Container $container) {
+        $this->share('aspect.pointcut.parser', function(Container $container) {
             return new PointcutParser(
                 new PointcutGrammar(
                     $container,

src/Instrument/Transformer/WeavingTransformer.php

Unused Use Statements -1 removed lines

@@ -18,7 +18,6 @@
 use Go\Core\AspectKernel;
 use Go\Core\AspectLoader;
 use Go\Instrument\ClassLoading\CachePathManager;
-use Go\Instrument\CleanableMemory;
 use Go\ParserReflection\ReflectionFile;
 use Go\ParserReflection\ReflectionFileNamespace;
 use Go\Proxy\ClassProxy;

src/Proxy/AbstractProxy.php

Doc Comments +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
     /**
      * Returns list of string representation of parameters
      *
-     * @param array|ReflectionParameter[] $parameters List of parameters
+     * @param ReflectionParameter[] $parameters List of parameters
      *
      * @return array
      */

Indentation +1 added lines, -1 removed lines

@@ -126,7 +126,7 @@
         $defaultValue = null;
         $isDefaultValueAvailable = $parameter->isDefaultValueAvailable();
         if ($isDefaultValueAvailable) {
-             $defaultValue = var_export($parameter->getDefaultValue(), true);
+                $defaultValue = var_export($parameter->getDefaultValue(), true);
         } elseif ($parameter->isOptional()) {
             $defaultValue = 'null';
         }

src/Proxy/ClassProxy.php

Spacing +3 added lines, -3 removed lines

@@ -416,7 +416,7 @@ 
         $scope    = $isStatic ? $this->staticLsbExpression : '$this';
         $prefix   = $isStatic ? AspectContainer::STATIC_METHOD_PREFIX : AspectContainer::METHOD_PREFIX;
 
-        $args = join(', ', array_map(function (ReflectionParameter $param) {
+        $args = join(', ', array_map(function(ReflectionParameter $param) {
             $byReference = $param->isPassedByReference() ? '&' : '';
 
             return $byReference . '$' . $param->name;
@@ -468,11 +468,11 @@ 
             ($prefix ? "$prefix " : '') . // List of class modifiers
             'class ' . // 'class' keyword with one space
             $this->name . // Name of the class
-            ' extends '. // 'extends' keyword with
+            ' extends ' . // 'extends' keyword with
             $this->parentClassName . // Name of the parent class
             ($this->interfaces ? ' implements ' . join(', ', $this->interfaces) : '') . "\n" . // Interfaces list
             "{\n" . // Start of class definition
-            ($this->traits ? $this->indent('use ' . join(', ', $this->traits) .';'."\n") : '') . "\n" . // Traits list
+            ($this->traits ? $this->indent('use ' . join(', ', $this->traits) . ';' . "\n") : '') . "\n" . // Traits list
             $this->indent(join("\n", $this->propertiesCode)) . "\n" . // Property definitions
             $this->indent(join("\n", $this->methodsCode)) . "\n" . // Method definitions
             "}" // End of class definition

src/Proxy/TraitProxy.php

Spacing +2 added lines, -2 removed lines

@@ -92,7 +92,7 @@ 
         $scope    = $isStatic ? $this->staticLsbExpression : '$this';
         $prefix   = $isStatic ? AspectContainer::STATIC_METHOD_PREFIX : AspectContainer::METHOD_PREFIX;
 
-        $args = join(', ', array_map(function (ReflectionParameter $param) {
+        $args = join(', ', array_map(function(ReflectionParameter $param) {
             $byReference = $param->isPassedByReference() ? '&' : '';
 
             return $byReference . '$' . $param->name;
@@ -123,7 +123,7 @@ 
                 'use ' . join(', ', array(-1 => $this->parentClassName) + $this->traits) .
                 $this->getMethodAliasesCode()
             ) . "\n" . // Use traits and aliases section
-            $this->indent(join("\n", $this->methodsCode)) . "\n". // Method definitions
+            $this->indent(join("\n", $this->methodsCode)) . "\n" . // Method definitions
             "}" // End of trait body
         );