Skip to content

Item Comparator#1

Open
Banarim wants to merge 1 commit into
dotintent:masterfrom
Banarim:item-comparator
Open

Item Comparator#1
Banarim wants to merge 1 commit into
dotintent:masterfrom
Banarim:item-comparator

Conversation

@Banarim

@Banarim Banarim commented Oct 26, 2016

Copy link
Copy Markdown

From now there is possibility to add comparator for data items not only for sections

@NonNull
private List<Section<T>> itemsToSections(@Nullable Collection<T> items,
@NonNull Comparator<Section> comparator,
@NonNull Comparator<T> dataObjectComparator) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use elvis, as before

private List<Section<T>> itemsToSections(@Nullable Collection<T> items,
@NonNull Comparator<Section> comparator,
@NonNull Comparator<T> dataObjectComparator) {
if (items != null) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inline this method

@Banarim Banarim force-pushed the item-comparator branch 3 times, most recently from a1bbd46 to 3eb71f7 Compare October 26, 2016 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants