Uploaded image for project: 'GMAT'
  1. GMAT
  2. GMT-6350

Line Highlighting Wrap-up: Use GuiInterpreter for ScriptInterpreter tasks

    Details

      Description

      The error reporting from the GUI interpreter, when used for script parsing, is different from the error reporting for the script interpreter. The Debug environment needs to access information from the interpret process performed when the script is parsed, so we need to either

      1. Update the GuiInterpreter class to set the messages consistently with the ScriptInterpreter, or
      2. Provide access to the ScriptInterpreter results for use in the script editor.

      The needed data is a list of line numbers where errors (and warnings) were found.

      Note: We might be able to do this using static lists (std::vectors) on the Interpreter base class.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              djcinsb Darrel Conway added a comment -

              We reworked the calling sequence for the separate interpreters instead.

              Show
              djcinsb Darrel Conway added a comment - We reworked the calling sequence for the separate interpreters instead.
              Hide
              shughes Steven Hughes added a comment -

              One slightly ugly issue: if you load a script that has errors, if you edit the script by adding a line, or removing a line, the line highlighting continues to highlight the old lines... which are now incorrect.

              It appears to me that Visual Studio is performing dynamic interpreting. Not sure what, if anything we should do, wanted to document to get your thoughts.

              Show
              shughes Steven Hughes added a comment - One slightly ugly issue: if you load a script that has errors, if you edit the script by adding a line, or removing a line, the line highlighting continues to highlight the old lines... which are now incorrect. It appears to me that Visual Studio is performing dynamic interpreting. Not sure what, if anything we should do, wanted to document to get your thoughts.

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Time Tracking

                    Estimated:
                    Original Estimate - 6 hours
                    6h
                    Remaining:
                    Remaining Estimate - 6 hours
                    6h
                    Logged:
                    Time Spent - Not Specified
                    Not Specified