[LANG-1674] Range construction with abstract/derived classes

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.13.0
• Assignee: Unassigned
• Reporter: Baljit Singh
• Created: 05/Oct/21 14:41
• Updated: 03/Apr/22 16:55
• Resolved: 17/Oct/21 13:35

Description

The constructors for org.apache.commons.lang3.Range don't play nicely with abstract/derived classes. Example:

abstract class AbstractComparable implements Comparable<AbstractComparable> {
@Override public int compareTo(AbstractComparable o) {}
}

class DerivedA extends AbstractComparable {}

class DerivedB extends AbstractComparable {}

// compiles, and reasonable since AbstractComparable is the common parent
static final Range<AbstractComparable> RANGE_MIXED = Range.between(new DerivedA(), new DerivedB());

// compiles, but unreasonable to force usage of parent class
static final Range<AbstractComparable> RANGE_SAME_CLASS = Range.between(new DerivedA(), new DerivedA());

// compiler error
static final Range<DerivedA> RANGE_A = Range.between(new DerivedA(), new DerivedA());

// compiler error
static final Range<DerivedB> RANGE_B = Range.is(new DerivedB());