Uploaded image for project: 'Commons JEXL'
  1. Commons JEXL
  2. JEXL-164

public getters for high/low properties for IntegerRange and LongRange

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Fixed
    • 3.0
    • 3.0
    • None

    Description

      Provide public getters for high/low properties for IntegerRange and LongRange

      Attachments

        Activity

          dmitri_blinov Dmitri Blinov added a comment -

          Can we also use org.apache.commons.lang3.Range<T> as a base for IntegerRange and LongRange?

          dmitri_blinov Dmitri Blinov added a comment - Can we also use org.apache.commons.lang3.Range<T> as a base for IntegerRange and LongRange?
          henrib Henri Biestro added a comment -

          Let's not add a dependency to apache commons lang3 for this purpose.
          However, you can derive JexlArithmetic and override method 'createRange' so it creates a Range<T> .

          henrib Henri Biestro added a comment - Let's not add a dependency to apache commons lang3 for this purpose. However, you can derive JexlArithmetic and override method 'createRange' so it creates a Range<T> .
          henrib Henri Biestro added a comment -

          added getMin/getMax methods, added tests

          src/main/java/org/apache/commons/jexl3/internal/IntegerRange.java
          src/main/java/org/apache/commons/jexl3/internal/LongRange.java
          src/site/xdoc/changes.xml
          src/test/java/org/apache/commons/jexl3/internal/RangeTest.java

          Revision: 1691507

          henrib Henri Biestro added a comment - added getMin/getMax methods, added tests src/main/java/org/apache/commons/jexl3/internal/IntegerRange.java src/main/java/org/apache/commons/jexl3/internal/LongRange.java src/site/xdoc/changes.xml src/test/java/org/apache/commons/jexl3/internal/RangeTest.java Revision: 1691507
          dmitri_blinov Dmitri Blinov added a comment -

          I'll give it a try, thanks for the hint and for your efforts!

          dmitri_blinov Dmitri Blinov added a comment - I'll give it a try, thanks for the hint and for your efforts!
          henrib Henri Biestro added a comment -

          Update issues resolved in 3.0 to closed

          henrib Henri Biestro added a comment - Update issues resolved in 3.0 to closed

          People

            henrib Henri Biestro
            dmitri_blinov Dmitri Blinov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: