Uploaded image for project: 'Jalview'
  1. Jalview
  2. JAL-3270

ColourSchemes enum requires loading of every color scheme

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: jalview-j2s
    • Labels:
      None

      Description

      Right now JalviewColourScheme loads every possible class as part of its initialization. wouldn't it better to do this in a lazy fashion? Just declare the value as the name, and then initialize upon getSchemeClass() calling. Registration in ColourSchemes, I think, could also be done lazily by using the enum constant's name instead of the ColourScheme's name (which are by definition the same). Looks like mostly the name is checked anyway,

      Maybe not worth the trouble.

        Attachments

          Activity

            People

            Assignee:
            gmungoc Mungo Carstairs
            Reporter:
            hansonr Bob Hanson
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated: