Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-128423

MethodInfo refactoring

    XMLWordPrintable

Details

    • Suggestion
    • Resolution: Unresolved
    • P3: Somewhat important
    • Some future release
    • None
    • QML: Tooling
    • None
    • 67ab9e5a1 (dev), 1186f254f (6.8), 0ba440a29 (6.7), 751beaff2 (dev), 49cc94822 (dev)

    Description

      Differentiating between Signals and Methods at the DOM API level, using kindValue (DomType), can be beneficial in certain contexts, such as the qmlls DocumentSymbol request (Outline). The formatting also varies significantly based on MethodType, with notable differences between Signals and Methods. Since Signals are fundamentally distinct constructs from Methods, some cases, like the qmlls DocumentSymbols request, could benefit from having their own "available" signature handling.

      These points raise several considerations:

      • Should Signals and Methods be separated? For instance, should we keep only method/function-related elements within MethodInfo and move Signal-related functionality to a dedicated Signal structure?
      • Should we introduce a unified mechanism for storing and retrieving signatures? (For example adding a new member to MethodInfo, which is filled by QQmlDomAstCreator)
      • If Signals and Methods remain under MethodInfo,  we  should better unify / refactor output methods (e.g., writeOut) and parameter handling
      • Do Signals and Methods have more differences than similarities, suggesting the need for a clearer separation?

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              qtqmlteam Qt Qml Team User
              dima.a Dmitrii Akshintsev
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes