This is an automated email from the ASF dual-hosted git repository. paulk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/groovy.git
commit f0f2153b4dce9fcae94eb1fdbf3b91828a2fd9fa Author: John Dong <[email protected]> AuthorDate: Sat May 3 20:21:07 2025 +0800 add partitionPoint to Groovy Array and List - fix comment --- .../groovy/runtime/ArrayGroovyMethods.java | 35 +++++++++++++--------- .../groovy/runtime/DefaultGroovyMethods.java | 5 ++-- 2 files changed, 24 insertions(+), 16 deletions(-) diff --git a/src/main/java/org/codehaus/groovy/runtime/ArrayGroovyMethods.java b/src/main/java/org/codehaus/groovy/runtime/ArrayGroovyMethods.java index 39fd1f2719..bba5dcd8c4 100644 --- a/src/main/java/org/codehaus/groovy/runtime/ArrayGroovyMethods.java +++ b/src/main/java/org/codehaus/groovy/runtime/ArrayGroovyMethods.java @@ -6599,8 +6599,9 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint(0..arr.size()) { it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint(0..arr.size()) { it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint(0..arr.size()) { it <= 0 } == 0 + * //for none match condition with range * assert arr.partitionPoint(2..arr.size()) { it <= 0 } == 2 * </pre> * @@ -6642,9 +6643,9 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * //usage case as upperBound(cpp), bisect_right(python) * assert arr.partitionPoint{ it <= 4 } == 6 * //for all match condition - * assert arr.partitionPoint{ it <= 100 } == arr.size() - * //for all match condition - * assert arr.partitionPoint{ it <= 0 } == 0 + * assert arr.partitionPoint(0..arr.size()) { it <= 100 } == arr.size() + * //for none match condition + * assert arr.partitionPoint(0..arr.size()) { it <= 0 } == 0 * </pre> * * @param self a groovy arr @@ -6673,8 +6674,9 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint(0..arr.size()) { it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint(0..arr.size()) { it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint(0..arr.size()) { it <= 0 } == 0 + * //for none match condition with range * assert arr.partitionPoint(2..arr.size()) { it <= 0 } == 2 * </pre> * @@ -6716,7 +6718,7 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint{ it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint{ it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint{ it <= 0 } == 0 * </pre> * @@ -6746,8 +6748,9 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint(0..arr.size()) { it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint(0..arr.size()) { it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint(0..arr.size()) { it <= 0 } == 0 + * //for none match condition with range * assert arr.partitionPoint(2..arr.size()) { it <= 0 } == 2 * </pre> * @@ -6789,7 +6792,7 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint{ it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint{ it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint{ it <= 0 } == 0 * </pre> * @@ -6821,6 +6824,7 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint(0..arr.size()) { it <= 100 } == arr.size() * //for all match condition * assert arr.partitionPoint(0..arr.size()) { it <= 0 } == 0 + * //for none match condition with range * assert arr.partitionPoint(2..arr.size()) { it <= 0 } == 2 * </pre> * @@ -6862,7 +6866,7 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint{ it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint{ it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint{ it <= 0 } == 0 * </pre> * @@ -6892,8 +6896,9 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint(0..arr.size()) { it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint(0..arr.size()) { it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint(0..arr.size()) { it <= 0 } == 0 + * //for none match condition with range * assert arr.partitionPoint(2..arr.size()) { it <= 0 } == 2 * </pre> * @@ -6935,7 +6940,7 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint{ it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint{ it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint{ it <= 0 } == 0 * </pre> * @@ -6965,8 +6970,9 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint(0..arr.size()) { it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint(0..arr.size()) { it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint(0..arr.size()) { it <= 0 } == 0 + * //for none match condition with range * assert arr.partitionPoint(2..arr.size()) { it <= 0 } == 2 * </pre> * @@ -7008,7 +7014,7 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint{ it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint{ it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint{ it <= 0 } == 0 * </pre> * @@ -7040,6 +7046,7 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint(0..arr.size()) { it <= 100 } == arr.size() * //for all match condition * assert arr.partitionPoint(0..arr.size()) { it <= 0 } == 0 + * //for none match condition with range * assert arr.partitionPoint(2..arr.size()) { it <= 0 } == 2 * </pre> * @@ -7081,7 +7088,7 @@ public class ArrayGroovyMethods extends DefaultGroovyMethodsSupport { * assert arr.partitionPoint{ it <= 4 } == 6 * //for all match condition * assert arr.partitionPoint{ it <= 100 } == arr.size() - * //for all match condition + * //for none match condition * assert arr.partitionPoint{ it <= 0 } == 0 * </pre> * diff --git a/src/main/java/org/codehaus/groovy/runtime/DefaultGroovyMethods.java b/src/main/java/org/codehaus/groovy/runtime/DefaultGroovyMethods.java index a59e986b90..56dc038547 100644 --- a/src/main/java/org/codehaus/groovy/runtime/DefaultGroovyMethods.java +++ b/src/main/java/org/codehaus/groovy/runtime/DefaultGroovyMethods.java @@ -11068,8 +11068,9 @@ public class DefaultGroovyMethods extends DefaultGroovyMethodsSupport { * assert list.partitionPoint(0..list.size()) { it <= 4 } == 6 * //for all match condition * assert list.partitionPoint(0..list.size()) { it <= 100 } == list.size() - * //for all match condition + * //for none match condition * assert list.partitionPoint(0..list.size()) { it <= 0 } == 0 + * //for none match condition with range * assert list.partitionPoint(2..list.size()) { it <= 0 } == 2 * </pre> * @@ -11111,7 +11112,7 @@ public class DefaultGroovyMethods extends DefaultGroovyMethodsSupport { * assert list.partitionPoint{ it <= 4 } == 6 * //for all match condition * assert list.partitionPoint{ it <= 100 } == list.size() - * //for all match condition + * //for none match condition * assert list.partitionPoint{ it <= 0 } == 0 * </pre> *
