Uploaded image for project: 'Apache Flex'
  1. Apache Flex
  2. FLEX-19388

Cleanup: WindowedSystemManager has lots of dead code because topLevel is never false

    XMLWordPrintableJSON

Details

    • Task
    • Status: Closed
    • Trivial
    • Resolution: Later
    • Adobe Flex SDK 3.1 (Release)
    • None
    • mx: Window
    • None
    • Affected OS(s): All OS Platforms
      Language Found: English

    Description

      WindowedSystemManager has a topLevel property that is always true (it's private, initialized to true, and never set anywhere else). There's a lot of code with if statements checking the value of topLevel, and a good amount of those blocks of code will only run if topLevel is false (which it never will be). I think all of this is vestigal, from when WindowedSystemManager started out by copy & pasting all the SystemManager code in bulk.

      We should clean this up – remove all the code that will never run, and remove the topLevel property itself.

      (This cruft still exists in Flex 4, btw)

      Attachments

        Activity

          People

            adobejira Adobe JIRA
            adobejira Adobe JIRA
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: