Clirr reports:

Parameter 1 of 'public org.apache.commons.net.nntp.Threadable
thread(org.apache.commons.net.nntp.Threadable[])' has changed its type
to java.util.List

It looks like this may have been an accidental change; it was
committed as part of  r761877 whose log message is:

* Improve handling of invalid messages
* Add TODO for converting recursive flush() to iterative version

AFAICT, the change to Threadable does not fit with the rest of the commit.

I think the change should either be reverted, or the old API should be
reinstated and deprecated, for example using:

    public Threadable thread(Threadable[] messages) {
        return thread(Arrays.asList(messages));
    }


I agree that using a List is better, but the API should not be changed
in a minor release.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to