Skip to content

Commit 33d81cf

Browse files
committed
Remove unused variables
1 parent d89aef8 commit 33d81cf

File tree

5 files changed

+4
-6
lines changed

5 files changed

+4
-6
lines changed

src/main/QafooLabs/Refactoring/Domain/Model/PhpName.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,6 @@ private function overlaps(PhpName $other)
8787

8888
private function shareNamespace(PhpName $other)
8989
{
90-
$otherName = array();
9190
$otherParts = $this->stringToParts($other->fullyQualifiedName);
9291

9392
return strpos($this->fullyQualifiedName, $otherParts[0]) !== false;

src/test/QafooLabs/Refactoring/Application/RenameLocalVariableTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public function testRenameLocalVariable()
2929
\Phake::when($this->editor)->openBuffer(\Phake::anyParameters())->thenReturn($buffer);
3030
\Phake::when($this->codeAnalysis)->findMethodRange(\Phake::anyParameters())->thenReturn(LineRange::fromSingleLine(1));
3131

32-
$patch = $this->refactoring->refactor(new File('foo.php', <<<'PHP'
32+
$this->refactoring->refactor(new File('foo.php', <<<'PHP'
3333
<?php
3434
class Foo
3535
{

src/test/QafooLabs/Refactoring/Application/Service/ExtractMethodTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ public function setUp()
2828
*/
2929
public function testRefactorSimpleMethod()
3030
{
31-
$patch = $this->refactoring->refactor(new File('foo.php', <<<'PHP'
31+
$this->refactoring->refactor(new File('foo.php', <<<'PHP'
3232
<?php
3333
class Foo
3434
{
@@ -69,7 +69,7 @@ public function testVariableUsedBeforeAndAfterExtractedSlice()
6969
{
7070
$this->markTestIncomplete('Failing over some invisible whitespace issue?');
7171

72-
$patch = $this->refactoring->refactor(new File('foo.php', <<<'PHP'
72+
$this->refactoring->refactor(new File('foo.php', <<<'PHP'
7373
<?php
7474
class Foo
7575
{

src/test/QafooLabs/Refactoring/Domain/Model/MethodSignatureTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public function whenCreateMethodSignatureWithInvalidVisibility_ThenThrowExceptio
2222
{
2323
$this->setExpectedException('InvalidArgumentException');
2424

25-
$method = new MethodSignature('foo', MethodSignature::IS_PRIVATE | MethodSignature::IS_PUBLIC);
25+
new MethodSignature('foo', MethodSignature::IS_PRIVATE | MethodSignature::IS_PUBLIC);
2626
}
2727

2828
/**

src/test/QafooLabs/Refactoring/Domain/Model/PhpNameTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,6 @@ public function testRegression4()
9595
{
9696
$name = new PhpName('Foo', 'Foo');
9797
$from = new PhpName('Foo\\Foo', 'Foo');
98-
$to = new PhpName('Foo\\Bar', 'Bar');
9998

10099
$this->assertFalse($name->isAffectedByChangesTo($from), 'Namespace should not be affected by changes to Class in namespace.');
101100
}

0 commit comments

Comments
 (0)