Skip to content

Commit

Permalink
Cleanup AlignMultilineParametersFixer comments
Browse files Browse the repository at this point in the history
  • Loading branch information
erickskrauch committed Jan 8, 2024
1 parent 0f9fc91 commit ec69923
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions src/FunctionNotation/AlignMultilineParametersFixer.php
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ public function isCandidate(Tokens $tokens): bool {

/**
* Must run after StatementIndentationFixer, MethodArgumentSpaceFixer, CompactNullableTypehintFixer,
* SingleSpaceAroundConstructFixer
* SingleSpaceAroundConstructFixer, TypesSpacesFixer
*/
public function getPriority(): int {
return -10;
Expand Down Expand Up @@ -197,10 +197,6 @@ protected function applyFix(SplFileInfo $file, Tokens $tokens): void {
}
}

/**
* TODO: The declaration might be split across multiple lines.
* In such case we need to find the longest line and return it as the full type length
*/
private function getFullTypeLength(Tokens $tokens, TypeAnalysis $typeAnalysis): int {
$typeLength = 0;
for ($i = $typeAnalysis->getStartIndex(); $i <= $typeAnalysis->getEndIndex(); $i++) {
Expand Down

0 comments on commit ec69923

Please sign in to comment.