Merge pull request #133 from phpDocumentor/benchmark

Benchmark

Author: jaapio

.travis.yml

@@ -1,5 +1,5 @@
 language: php
-php: [ 7.1, 7.2, nightly ]
+php: [ 7.1, 7.2, 7.3, nightly ]
 sudo: false
 
 env:

composer.json

@@ -14,7 +14,7 @@
             "phpDocumentor\\": [
                 "tests/component/",
                 "tests/unit/phpDocumentor",
-                "tests/mocks/"
+                "tests/bench/"
             ]
         }
     },
@@ -27,7 +27,7 @@
     },
     "require-dev": {
         "mockery/mockery": "~1.0",
-        "mikey179/vfsStream": "~1.2"
+        "mikey179/vfsstream": "~1.2"
     },
     "extra": {
         "branch-alias": {

composer.lock

@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phive xmlns="https://phar.io/phive">
   <phar name="phpstan" version="^0.9.1" installed="0.10.7" location="./tools/phpstan" copy="true"/>
-  <phar name="phpunit" version="^6.0" installed="6.5.13" location="./tools/phpunit" copy="true"/>
+  <phar name="phpunit" version="^7.0" installed="7.5.7" location="./tools/phpunit" copy="true"/>
+  <phar name="phpbench" version="^0.16.9" installed="0.16.9" location="./tools/phpbench" copy="true"/>
 </phive>

phive.xml

@@ -0,0 +1,7 @@
+{
+  "bootstrap": "vendor/autoload.php",
+  "path": "tests/bench",
+  "extensions": [
+    "PhpBench\\Extensions\\XDebug\\XDebugExtension"
+  ]
+}

phpbench.json

@@ -26,6 +26,7 @@
 use phpDocumentor\Reflection\Php\StrategyContainer;
 use phpDocumentor\Reflection\Types\Context;
 use phpDocumentor\Reflection\Types\ContextFactory;
+use phpDocumentor\Reflection\Types\NamespaceNodeToContext;
 use PhpParser\Comment\Doc;
 use PhpParser\Node;
 use PhpParser\Node\Stmt\Class_ as ClassNode;
@@ -98,10 +99,7 @@ private function createFile(CreateCommand $command)
         $code = $file->getContents();
         $nodes = $this->nodesFactory->create($code);
 
-        $contextFactory = new ContextFactory();
-        $context = $contextFactory->createForNamespace('\\', $code);
-
-        $docBlock = $this->createFileDocBlock(null, $command->getStrategies(), $context, $nodes);
+        $docBlock = $this->createFileDocBlock(null, $command->getStrategies(), null, $nodes);
 
         $result = new FileElement(
             $file->md5(),
@@ -110,18 +108,20 @@ private function createFile(CreateCommand $command)
             $docBlock
         );
 
-        $this->createElements(new Fqsen('\\'), $nodes, $result, $command->getStrategies());
+        $this->createElements($nodes, $result, $command->getStrategies(), null);
 
         return $result;
     }
 
     /**
      * @param Node[] $nodes
      */
-    private function createElements(Fqsen $namespace, array $nodes, FileElement $file, StrategyContainer $strategies): void
-    {
-        $contextFactory = new ContextFactory();
-        $context = $contextFactory->createForNamespace((string) $namespace, $file->getSource());
+    private function createElements(
+        array $nodes,
+        FileElement $file,
+        StrategyContainer $strategies,
+        ?Context $context
+    ): void {
         foreach ($nodes as $node) {
             switch (get_class($node)) {
                 case ClassNode::class:
@@ -140,8 +140,9 @@ private function createElements(Fqsen $namespace, array $nodes, FileElement $fil
                     $file->addInterface($interface);
                     break;
                 case NamespaceNode::class:
+                    $context = (new NamespaceNodeToContext())($node);
                     $file->addNamespace($node->fqsen);
-                    $this->createElements($node->fqsen, $node->stmts, $file, $strategies);
+                    $this->createElements($node->stmts, $file, $strategies, $context);
                     break;
                 case TraitNode::class:
                     $strategy = $strategies->findMatching($node);