From fdbc067027810fe767af9d4b534caa97eac20dce Mon Sep 17 00:00:00 2001 From: Philip Cohn-Cort Date: Fri, 26 Oct 2018 15:33:28 -0400 Subject: [PATCH] [Refactor] - Use prebuilt `Rect::setEmpty` instead of `Rect::set(0,0,0,0)` --- .../main/kotlin/edu/artic/decoration/AllToursItemDecoration.kt | 2 +- .../main/java/edu/artic/search/SearchDividerItemDecoration.kt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/content_listing/src/main/kotlin/edu/artic/decoration/AllToursItemDecoration.kt b/content_listing/src/main/kotlin/edu/artic/decoration/AllToursItemDecoration.kt index 44a7a7af1..be904c973 100644 --- a/content_listing/src/main/kotlin/edu/artic/decoration/AllToursItemDecoration.kt +++ b/content_listing/src/main/kotlin/edu/artic/decoration/AllToursItemDecoration.kt @@ -21,7 +21,7 @@ class AllToursItemDecoration( override fun getItemOffsets(outRect: Rect, view: View, parent: RecyclerView, state: RecyclerView.State) { val adjustedPos = parent.getChildAdapterPosition(view) - 1 // item position, minus 1 for the header if (adjustedPos < 0) { - outRect.set(0, 0, 0, 0) + outRect.setEmpty() } else { val column = (adjustedPos) % spanCount // item column diff --git a/search/src/main/java/edu/artic/search/SearchDividerItemDecoration.kt b/search/src/main/java/edu/artic/search/SearchDividerItemDecoration.kt index 4f057dead..fef2c3cbd 100644 --- a/search/src/main/java/edu/artic/search/SearchDividerItemDecoration.kt +++ b/search/src/main/java/edu/artic/search/SearchDividerItemDecoration.kt @@ -58,7 +58,7 @@ class SearchDividerItemDecoration(context: Context) : RecyclerView.ItemDecoratio } else if ((it is SearchBaseListItemViewModel || it.hasDivider) && (position < adapter.itemCount - 1 )) { outRect.set(0, 0, 0, mDividerHeight) } else { - outRect.set(0, 0, 0, 0) + outRect.setEmpty() } } }