onLoadMoreComplete(null) call from noMoreLoad() cause infinite loop

According to documentation of onLoadMoreComplete(@Nullable List<T> newItems)

When noMoreLoad OR onError OR onCancel, pass empty list or null to hide the progressItem.

If null or empty list is passed in onLoadMoreComplete() the last if clause is executed and noMoreLoad is called again.

v5.0.0-rc1

public void onLoadMoreComplete(@Nullable List<T> newItems, @IntRange(from = -1) long delay) {
		// 1. Calculate new items count
		int newItemsSize = newItems == null ? 0 : newItems.size();
		int totalItemCount = newItemsSize + getMainItemCount();
		// 2. Add any new items
		if (newItemsSize > 0) {
			if (DEBUG)
				Log.v(TAG, "onLoadMore     performing adding " + newItemsSize + " new items on Page=" + getEndlessCurrentPage());
			//TODO: 0 + headers for Endless Top Scrolling
			addItems(getGlobalPositionOf(mProgressItem), newItems);
		}
		// 3. Check if features are enabled and the limits have been reached
		if (mEndlessPageSize > 0 && newItemsSize < mEndlessPageSize || // Is feature enabled and Not enough items?
				mEndlessTargetCount > 0 && totalItemCount >= mEndlessTargetCount) { // Is feature enabled and Max limit has been reached?
			// Disable the EndlessScroll feature
			setEndlessProgressItem(null);
		}
		// 4. Remove the progressItem if needed
		if (delay > 0 && (newItemsSize == 0 || !isEndlessScrollEnabled())) {
			if (DEBUG)
				Log.v(TAG, "onLoadMore     enqueued removing progressItem (" + delay + "ms)");
			mHandler.sendEmptyMessageDelayed(LOAD_MORE_COMPLETE, delay);
		} else if (isEndlessScrollEnabled()) {
			hideProgressItem();
		}
		// 5. Reset the loading status
		endlessLoading = false;
		// 6. Eventually notify noMoreLoad
		if (newItemsSize == 0 || !isEndlessScrollEnabled()) {
			noMoreLoad(newItemsSize);
		}
	}

[KyrychenkoOleksandr]

@davideas progressItem is not automatically hidden inside void onLoadMoreComplete(@Nullable List<T> newItems, @IntRange(from = -1) long delay)

// 3. Check if features are enabled and the limits have been reached
		if (mEndlessPageSize > 0 && newItemsSize < mEndlessPageSize || // Is feature enabled and Not enough items?
				mEndlessTargetCount > 0 && totalItemCount >= mEndlessTargetCount) { // Is feature enabled and Max limit has been reached?
			// Disable the EndlessScroll feature
			setEndlessProgressItem(null);
		}

setEndlessProgressItem(null); is called. inside

public FlexibleAdapter setEndlessProgressItem(@Nullable T progressItem) {
		endlessScrollEnabled = progressItem != null;
		if (progressItem != null) {
			setEndlessScrollThreshold(mEndlessScrollThreshold);
			mProgressItem = progressItem;
			if (DEBUG) {
				Log.i(TAG, "Set progressItem=" + getClassName(progressItem));
				Log.i(TAG, "Enabled EndlessScrolling");
			}
		} else if (DEBUG) {
			Log.i(TAG, "Disabled EndlessScrolling");
		}
		return this;
	}

endlessScrollEnabled is set to false. than inside void onLoadMoreComplete(@Nullable List<T> newItems, @IntRange(from = -1) long delay)

// 4. Remove the progressItem if needed
		if (delay > 0 && (newItemsSize == 0 || !isEndlessScrollEnabled())) {
			if (DEBUG)
				Log.v(TAG, "onLoadMore     enqueued removing progressItem (" + delay + "ms)");
			mHandler.sendEmptyMessageDelayed(LOAD_MORE_COMPLETE, delay);
		} else if (isEndlessScrollEnabled()) {
			hideProgressItem();
		}

hideProgressItem(); can’t be called because isEndlessScrollEnabled() returns false.

[davideas]

@eemihauk, removing the else condition should then work also when no items and no delay.