Uploaded image for project: 'Commons Lang'
  1. Commons Lang
  2. LANG-514

CharRange constructor is hard to read

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.5
    • lang.*
    • None

    Description

      Saying new CharRange("....", true) is a very painful way to negate the '...'. Both for cryptic, and because you'd expect true to mean positiveness and false to imply the 'not'.

      Replace this with a static method:

      CharRange.not("....").

      Attachments

        1. LANG-514.patch
          2 kB
          Vincent Ricard
        2. LANG-514-private-ctor.patch
          35 kB
          Vincent Ricard

        Activity

          People

            Unassigned Unassigned
            bayard Henri Yandell
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: