From 54d2e28226405eb05d69103eb6307dad82e56a19 Mon Sep 17 00:00:00 2001 From: Siebrand Mazeland Date: Mon, 21 Apr 2014 19:48:22 +0200 Subject: [PATCH] Update formatting for languages/utils/ Change-Id: I54a19cff97067b71cb6531bc37802adb2d6c52ba --- languages/utils/CLDRPluralRuleConverter.php | 13 ++++++++++--- .../utils/CLDRPluralRuleConverterExpression.php | 2 +- languages/utils/CLDRPluralRuleConverterFragment.php | 1 - languages/utils/CLDRPluralRuleConverterOperator.php | 2 +- languages/utils/CLDRPluralRuleError.php | 3 +-- languages/utils/CLDRPluralRuleEvaluator.php | 7 ++++++- languages/utils/CLDRPluralRuleEvaluatorRange.php | 3 ++- 7 files changed, 21 insertions(+), 10 deletions(-) diff --git a/languages/utils/CLDRPluralRuleConverter.php b/languages/utils/CLDRPluralRuleConverter.php index 50a22c9c1b..1294b1a06b 100644 --- a/languages/utils/CLDRPluralRuleConverter.php +++ b/languages/utils/CLDRPluralRuleConverter.php @@ -1,5 +1,4 @@ doConvert(); } @@ -192,6 +192,7 @@ class CLDRPluralRuleConverter { if ( $length !== 0 ) { $token = $this->newNumber( substr( $this->rule, $this->pos, $length ), $this->pos ); $this->pos += $length; + return $token; } @@ -200,6 +201,7 @@ class CLDRPluralRuleConverter { if ( $op2 === '..' || $op2 === '!=' ) { $token = $this->newOperator( $op2, $this->pos, 2 ); $this->pos += 2; + return $token; } @@ -207,7 +209,8 @@ class CLDRPluralRuleConverter { $op1 = $this->rule[$this->pos]; if ( $op1 === ',' || $op1 === '=' || $op1 === '%' ) { $token = $this->newOperator( $op1, $this->pos, 1 ); - $this->pos ++; + $this->pos++; + return $token; } @@ -235,6 +238,7 @@ class CLDRPluralRuleConverter { if ( isset( self::$precedence[$bothWords] ) ) { $token = $this->newOperator( $bothWords, $this->pos, $nextTokenPos - $this->pos ); $this->pos = $nextTokenPos; + return $token; } } @@ -243,13 +247,15 @@ class CLDRPluralRuleConverter { if ( isset( self::$precedence[$word1] ) ) { $token = $this->newOperator( $word1, $this->pos, strlen( $word1 ) ); $this->pos += strlen( $word1 ); + return $token; } // The single-character operand symbols if ( strpos( self::OPERAND_SYMBOLS, $word1 ) !== false ) { $token = $this->newNumber( $word1, $this->pos ); - $this->pos ++; + $this->pos++; + return $token; } @@ -258,6 +264,7 @@ class CLDRPluralRuleConverter { // Samples are like comments, they have no effect on rule evaluation. // They run from the first sample indicator to the end of the string. $this->pos = $this->end; + return false; } diff --git a/languages/utils/CLDRPluralRuleConverterExpression.php b/languages/utils/CLDRPluralRuleConverterExpression.php index 2373807509..1ee6b4c54e 100644 --- a/languages/utils/CLDRPluralRuleConverterExpression.php +++ b/languages/utils/CLDRPluralRuleConverterExpression.php @@ -1,5 +1,4 @@ type ) { return true; } + return false; } } diff --git a/languages/utils/CLDRPluralRuleConverterFragment.php b/languages/utils/CLDRPluralRuleConverterFragment.php index 89ada4f28f..df299cbdf4 100644 --- a/languages/utils/CLDRPluralRuleConverterFragment.php +++ b/languages/utils/CLDRPluralRuleConverterFragment.php @@ -1,5 +1,4 @@ isType( $rightType ) ) { $newExpr->error( "invalid type for right operand: expected $rightType, got {$right->type}" ); } + return $newExpr; } } diff --git a/languages/utils/CLDRPluralRuleError.php b/languages/utils/CLDRPluralRuleError.php index 2ca3410dd2..cc0b5d2fb2 100644 --- a/languages/utils/CLDRPluralRuleError.php +++ b/languages/utils/CLDRPluralRuleError.php @@ -1,5 +1,4 @@ add( $right ); + return $range; case '..': return new CLDRPluralRuleEvaluatorRange( $left, $right ); diff --git a/languages/utils/CLDRPluralRuleEvaluatorRange.php b/languages/utils/CLDRPluralRuleEvaluatorRange.php index 8f8945a451..a62eeb98bc 100644 --- a/languages/utils/CLDRPluralRuleEvaluatorRange.php +++ b/languages/utils/CLDRPluralRuleEvaluatorRange.php @@ -55,6 +55,7 @@ class CLDRPluralRuleEvaluatorRange { } } } + return false; } @@ -102,7 +103,7 @@ class CLDRPluralRuleEvaluatorRange { } } $s .= ')'; + return $s; } - } -- 2.20.1