You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2022/10/18 07:00:00 UTC
[jira] [Work logged] (COLLECTIONS-814) CollectionUtils.removeAll() not throwing proper NullPointerException(NPE) if the first parameter is empty
[ https://issues.apache.org/jira/browse/COLLECTIONS-814?focusedWorklogId=817891&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-817891 ]
ASF GitHub Bot logged work on COLLECTIONS-814:
----------------------------------------------
Author: ASF GitHub Bot
Created on: 18/Oct/22 06:59
Start Date: 18/Oct/22 06:59
Worklog Time Spent: 10m
Work Description: angusdev commented on PR #340:
URL: https://github.com/apache/commons-collections/pull/340#issuecomment-1281900818
ping?
Issue Time Tracking
-------------------
Worklog Id: (was: 817891)
Time Spent: 0.5h (was: 20m)
> CollectionUtils.removeAll() not throwing proper NullPointerException(NPE) if the first parameter is empty
> ---------------------------------------------------------------------------------------------------------
>
> Key: COLLECTIONS-814
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-814
> Project: Commons Collections
> Issue Type: Bug
> Affects Versions: 4.4
> Reporter: Elia Bertolina
> Priority: Major
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> The CollectionUtils.removeAll(Collection<E> collection, Collection<?> remove) does not throw a NullPointerException(NPE) when the “remove” parameters is null, but only if the “collection” parameter is empty.
> In the documentation it is stated that an NPE will be thrown if any of the parameters is null.
> However, in this test case:
>
> {code:java}
> public class CollectionUtils_failure_Test {
> public void test() throws Throwable {
> LinkedList<InstanceofPredicate> linkedList = new
> LinkedList<InstanceofPredicate>();
> try {
> Collection<InstanceofPredicate> collection =
> CollectionUtils.removeAll(
> (Collection<InstanceofPredicate>)linkedList,
> (Collection<?>) null);
> org.junit.Assert.fail();
> } catch (java.lang.NullPointerException e) {
> //Exception caught and test successful
> }
> }
> } {code}
>
> This is a special case (first parameter needs to be empty and the second needs to be null) but this behavior is missing in the documentation. While this behavior is somehow correct (removing a null Object from an empty Collection we should obtain an empty Collection) I think throwing an NPE would be more in line with the documentation provided.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)