jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/332416 )

Change subject: readme: Fix example
......................................................................


readme: Fix example

Use $index instead of $i: the latter is incorrect and
causes "PHP Notice:  Undefined offset: 2".

Change-Id: I57f0317bf5180520a628968ac3ecfe5cd8407b3f
---
M README.md
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Umherirrender: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/README.md b/README.md
index 6f37ca9..c921565 100644
--- a/README.md
+++ b/README.md
@@ -18,7 +18,7 @@
 
 for ( $i = 0; $i < 3; $i++ ) {
        $index = Evaluator::evaluate( $i, $rules );
-       echo "This code has $i {$forms[$i]}\n";
+       echo "This code has $i {$forms[$index]}\n";
 }
 
 // This code has 0 syntax errors

-- 
To view, visit https://gerrit.wikimedia.org/r/332416
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I57f0317bf5180520a628968ac3ecfe5cd8407b3f
Gerrit-PatchSet: 1
Gerrit-Project: CLDRPluralRuleParser
Gerrit-Branch: master
Gerrit-Owner: Ricordisamoa <ricordisa...@openmailbox.org>
Gerrit-Reviewer: Umherirrender <umherirrender_de...@web.de>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to