IssueTracker

Component used: androidx.recyclerview:recyclerview Version used: 1.3.0

The documentation for removeOnScrollListener is incorrect.

It's currently:

    /**
     * Remove a listener that was notified of any changes in scroll state or position.
     *
     * @param listener listener to set or null to clear
     */

Problems:

• listener is not a listener to set, it's the one to remove.
• "that was notified of any changes in scroll state or position" is incorrect; the listener is removed irrespective of whether any scroll state or position changes occurred
• listener is not a nullable parameter, so the "or null to clear" is wrong.
• It doesn't describe what happens if listener was not previously added with addOnScrollListener.

Suggest rewriting to:

    /**
     * Remove a listener that was added with {@link #addOnScrollListener(OnScrollListener)}
     * 
     * <p>Does nothing if the listener was not previously added with
     * {@link #addOnScrollListener(OnScrollListener)}.</p>
     *
     * @param listener listener to remove
     */