Cleanup - squid:S1155 - Collection.isEmpty() should be used to test for emptiness (#371)

This commit is contained in:
Mohamed Ezzat 2016-06-29 15:31:41 +02:00 committed by inorichi
parent 05c0516a57
commit 4c1da3575b

View File

@ -530,7 +530,7 @@ public class VerticalViewPagerImpl extends ViewGroup {
setScrollingCacheEnabled(false);
return;
}
if (!always && mCurItem == item && mItems.size() != 0) {
if (!always && mCurItem == item && !mItems.isEmpty()) {
setScrollingCacheEnabled(false);
return;
}
@ -1676,7 +1676,7 @@ public class VerticalViewPagerImpl extends ViewGroup {
}
private boolean pageScrolled(int ypos) {
if (mItems.size() == 0) {
if (mItems.isEmpty()) {
mCalledSuper = false;
onPageScrolled(0, 0, 0);
if (!mCalledSuper) {
@ -2258,7 +2258,7 @@ public class VerticalViewPagerImpl extends ViewGroup {
targetPage = (int) (currentPage + pageOffset + truncator);
}
if (mItems.size() > 0) {
if (!mItems.isEmpty()) {
final ItemInfo firstItem = mItems.get(0);
final ItemInfo lastItem = mItems.get(mItems.size() - 1);
@ -2315,7 +2315,7 @@ public class VerticalViewPagerImpl extends ViewGroup {
super.onDraw(canvas);
// Draw the margin drawable between pages if needed.
if (mPageMargin > 0 && mMarginDrawable != null && mItems.size() > 0 && mAdapter != null) {
if (mPageMargin > 0 && mMarginDrawable != null && !mItems.isEmpty() && mAdapter != null) {
final int scrollY = getScrollY();
final int height = getHeight();