From 1102d5fb4eec9a712cd8f770dfa023e842abd830 Mon Sep 17 00:00:00 2001 From: Henrik Paul Date: Fri, 8 Aug 2014 09:55:22 +0300 Subject: Re-fixes "remove all rows" bug without breaking unit tests (#13334) This partly reverts a9fc5d5be Change-Id: I1d2f81e1d88c0057e6ba869358845ce4e9466a28 --- shared/src/com/vaadin/shared/ui/grid/Range.java | 6 ------ 1 file changed, 6 deletions(-) (limited to 'shared') diff --git a/shared/src/com/vaadin/shared/ui/grid/Range.java b/shared/src/com/vaadin/shared/ui/grid/Range.java index 4f7e37b9ad..2054845320 100644 --- a/shared/src/com/vaadin/shared/ui/grid/Range.java +++ b/shared/src/com/vaadin/shared/ui/grid/Range.java @@ -369,12 +369,6 @@ public final class Range implements Serializable { * if the two ranges aren't connected */ public Range combineWith(Range other) throws IllegalArgumentException { - if (other.isEmpty()) { - return this; - } else if (isEmpty()) { - return other; - } - if (getStart() > other.getEnd() || other.getStart() > getEnd()) { throw new IllegalArgumentException("There is a gap between " + this + " and " + other); -- cgit v1.2.3