Tim Starling has submitted this change and it was merged. Change subject: Fix various boundary cases in IcuCollation::findLowerBound() ......................................................................
Fix various boundary cases in IcuCollation::findLowerBound() Fix the following edge cases which were previously broken: * Zero-length input array * Target value before the start * Target value past the end They didn't really matter for my original application, but Liangent wants to use this function for something else. Change-Id: Ia5f5ed4ab3cb6c463177a4812fd3ce96c6d37b33 --- M includes/Collation.php 1 file changed, 13 insertions(+), 6 deletions(-) Approvals: Tim Starling: Verified; Looks good to me, approved jenkins-bot: Verified -- To view, visit https://gerrit.wikimedia.org/r/28491 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia5f5ed4ab3cb6c463177a4812fd3ce96c6d37b33 Gerrit-PatchSet: 3 Gerrit-Project: mediawiki/core Gerrit-Branch: chinese-collation Gerrit-Owner: Liangent <[email protected]> Gerrit-Reviewer: Liangent <[email protected]> Gerrit-Reviewer: Tim Starling <[email protected]> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
