Skip to content

Commit a6e89c9

Browse files
committed
Simplify property memoization for Flyweight pattern by replacing it with ??=
1 parent 50d4c64 commit a6e89c9

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

src/Parser/VariadicFunctionsVisitor.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,7 @@ public function beforeTraverse(array $nodes): ?array
4444
#[Override]
4545
public function enterNode(Node $node): ?Node
4646
{
47-
if ($this->topNode === null) {
48-
$this->topNode = $node;
49-
}
47+
$this->topNode ??= $node;
5048

5149
if ($node instanceof Node\Stmt\Namespace_ && $node->name !== null) {
5250
$this->inNamespace = $node->name->toString();

src/Parser/VariadicMethodsVisitor.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,7 @@ public function beforeTraverse(array $nodes): ?array
5353
#[Override]
5454
public function enterNode(Node $node): ?Node
5555
{
56-
if ($this->topNode === null) {
57-
$this->topNode = $node;
58-
}
56+
$this->topNode ??= $node;
5957

6058
if ($node instanceof Node\Stmt\Namespace_ && $node->name !== null) {
6159
$this->inNamespace = $node->name->toString();

0 commit comments

Comments
 (0)