Uploaded image for project: 'Groovy'
  1. Groovy
  2. GROOVY-3842

AST Browser - Browser should have option to show script, show class, or show both (performance improvement)

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7-beta-2
    • Fix Version/s: 1.7-rc-1
    • Component/s: Compiler
    • Labels:
      None
    • Environment:
      all

      Description

      The AST Browser in GroovyConsole shows the AST generated from the console script pane.

      The Browser shows both the free-standing script AST and the Script ClassNode ast (and any other Class AST).

      For most cases, the user wants to see either the script AST or the Class AST... but the browser must provide both. The browser is already slow to load for large scripts too.

      We can add a checkbox/radio button to the Browser so the user can pick to see the Script class, the script AST, or both. This will make the UI better for the user and it will also increase performance b/c the browser won't have to walk the tree twice.

        Attachments

        1. 3842_v17x.txt
          12 kB
          Roshan Dawrani
        2. AstBrowserScreenShotWithNewlyAddedUIOptions.JPG
          39 kB
          Roshan Dawrani

          Activity

            People

            • Assignee:
              roshandawrani Roshan Dawrani
              Reporter:
              hamletdrc Hamlet D'Arcy
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: