IssueTracker

the mChangesList.remove in the following code seems meaningless, because it will never be executed

public class DefaultItemAnimator extends SimpleItemAnimator {
    ...
    @Override
    public void endAnimations() {
        ...
        for (int i = listCount - 1; i >= 0; i--) {
            ArrayList<ChangeInfo> changes = mChangesList.get(i);
            count = changes.size();
            for (int j = count - 1; j >= 0; j--) {
                endChangeAnimationIfNecessary(changes.get(j));
                if (changes.isEmpty()) {
                    mChangesList.remove(changes);
                }
            }
        }
        ...
    }
    ...
}

I think may be this is the correct form

public class DefaultItemAnimator extends SimpleItemAnimator {
    ...
    @Override
    public void endAnimations() {
        ...
        for (int i = listCount - 1; i >= 0; i--) {
            ArrayList<ChangeInfo> changes = mChangesList.get(i);
            count = changes.size();
            for (int j = count - 1; j >= 0; j--) {
                endChangeAnimationIfNecessary(changes.get(j));
            }
            if (changes.isEmpty()) {
                mChangesList.remove(changes);
            }
        }
        ...
    }
    ...
}