Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8-beta-3
    • Fix Version/s: 1.7.6, 1.8-beta-3
    • Component/s: ast builder
    • Labels:
      None

      Description

      In AST Browser UI, if I change some parameters like Show Script flag (free form/class form) or Compile Phase and then press Refresh

      1) There is no proper progress indicator of the work that is happening in the background in the non-UI threads. The hourglass also appears if I move the mouse on the small area that shows the AST tree. In most of the remaining screen, mouse pointer looks normal. A progress indicator is needed.

      2) As soon as the refresh is pressed, the tree and property tables can be cleared as they take time to get populated and decompiled data comes up very fast and they look out-of-sync till the whole work is completed.

        Activity

        Roshan Dawrani created issue -
        Hide
        Hamlet D'Arcy added a comment -

        For issue 1 - This is correct behavior... the component that is loading has an hourglass but not the full window. The decompiler runs a lot faster than the tree view, so the decompiler comes up quick and you can work with it. This feature needs to remain in the browser. But you are correct... somehow the Loading message disappeared and we need to put a progress bar or loading tree node back in.

        It would be possible to create a progress bar based on the number of methods in every class in the AST, and then tick the progress bar forward once each time the ScriptToTreeNodeAdapter finds a method.

        Show
        Hamlet D'Arcy added a comment - For issue 1 - This is correct behavior... the component that is loading has an hourglass but not the full window. The decompiler runs a lot faster than the tree view, so the decompiler comes up quick and you can work with it. This feature needs to remain in the browser. But you are correct... somehow the Loading message disappeared and we need to put a progress bar or loading tree node back in. It would be possible to create a progress bar based on the number of methods in every class in the AST, and then tick the progress bar forward once each time the ScriptToTreeNodeAdapter finds a method.
        Hide
        Roshan Dawrani added a comment -

        Probably there is no need to come up with some complicated progress bar technique. Just a "Loading AST" message will be enough. Right now "loading" message is not correctly placed because the decompiler area gets refreshed much faster while most of the work is still going on.

        In http://groovyastbrowser.appspot.com/, as soon as refresh is pressed, I clear the AST in the tree and put a root node there that says "Loading"..may be something like that or a status bar message that remains there till the whole "refresh" operation is complete.

        Show
        Roshan Dawrani added a comment - Probably there is no need to come up with some complicated progress bar technique. Just a "Loading AST" message will be enough. Right now "loading" message is not correctly placed because the decompiler area gets refreshed much faster while most of the work is still going on. In http://groovyastbrowser.appspot.com/ , as soon as refresh is pressed, I clear the AST in the tree and put a root node there that says "Loading"..may be something like that or a status bar message that remains there till the whole "refresh" operation is complete.
        Hide
        Roshan Dawrani added a comment -

        The AST data population in the tree/table is very inconsistent. 9 out of 10 times, it is still remaining empty. I have waited for minutes for the worker threads to "suddenly" come

        Show
        Roshan Dawrani added a comment - The AST data population in the tree/table is very inconsistent. 9 out of 10 times, it is still remaining empty. I have waited for minutes for the worker threads to "suddenly" come
        Hide
        Roshan Dawrani added a comment -

        Hamlet, since the AST controls are still not correctly getting populated most of the times, and you are working on it, I am moving the issue to your name.

        Show
        Roshan Dawrani added a comment - Hamlet, since the AST controls are still not correctly getting populated most of the times, and you are working on it, I am moving the issue to your name.
        Roshan Dawrani made changes -
        Field Original Value New Value
        Assignee Guillaume Laforge [ guillaume ] Hamlet D'Arcy [ hamletdrc ]
        Hide
        Hamlet D'Arcy added a comment -

        fixed in 1.8

        Show
        Hamlet D'Arcy added a comment - fixed in 1.8
        Hamlet D'Arcy made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.8-beta-3 [ 16750 ]
        Resolution Fixed [ 1 ]
        Hide
        Hamlet D'Arcy added a comment -

        fixing on 1.7 branch

        Show
        Hamlet D'Arcy added a comment - fixing on 1.7 branch
        Hamlet D'Arcy made changes -
        Status Resolved [ 5 ] Reopened [ 4 ]
        Resolution Fixed [ 1 ]
        Hide
        Hamlet D'Arcy added a comment -

        fixing on 1.7 branch

        Show
        Hamlet D'Arcy added a comment - fixing on 1.7 branch
        Hamlet D'Arcy made changes -
        Fix Version/s 1.7.X [ 15538 ]
        Resolution Fixed [ 1 ]
        Status Reopened [ 4 ] Resolved [ 5 ]
        Paul King made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Paul King made changes -
        Resolution Fixed [ 1 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Paul King made changes -
        Fix Version/s 1.7.6 [ 16749 ]
        Fix Version/s 1.7.x [ 15538 ]
        Paul King made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 13:32:57 UTC 2015 [ 1428240777691 ]
        Mark Thomas made changes -
        Workflow jira [ 12733481 ] Default workflow, editable Closed status [ 12741042 ]
        Mark Thomas made changes -
        Project Import Mon Apr 06 02:11:23 UTC 2015 [ 1428286283443 ]
        Mark Thomas made changes -
        Workflow jira [ 12971264 ] Default workflow, editable Closed status [ 12979024 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        12h 30m 1 Hamlet D'Arcy 01/Dec/10 12:39
        Resolved Resolved Reopened Reopened
        2d 1h 45m 1 Hamlet D'Arcy 03/Dec/10 14:24
        Reopened Reopened Resolved Resolved
        9s 1 Hamlet D'Arcy 03/Dec/10 14:25
        Resolved Resolved Closed Closed
        12d 8h 6m 1 Paul King 15/Dec/10 22:31
        Closed Closed Reopened Reopened
        328d 6h 57m 1 Paul King 09/Nov/11 05:28
        Reopened Reopened Closed Closed
        21h 39m 1 Paul King 10/Nov/11 03:07

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development