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

Add screens in reverse order

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • P2: Important
    • 5.12.11, 5.13.0 Beta4
    • 5.11, 5.12.0
    • QPA: Wayland
    • None
    • All
    • 475910a75a5cd3332fe2f0e5740c4c3c2c0b8987 (qt/qtwayland/dev) b2ee9ff3992ee723be178c655464f2dc6f7a9b33 (qt/qtwayland/5.12)

    Description

      void QWaylandDisplay::registry_global(uint32_t id, const QString &interface, uint32_t version)
      {
          Q_UNUSED(version);
      
          struct ::wl_registry *registry = object();
      
          if (interface == QStringLiteral("wl_output")) {
              QWaylandScreen *screen = new QWaylandScreen(this, version, id);
              mScreens.append(screen);
              // We need to get the output events before creating surfaces
              forceRoundTrip();
      #ifndef Q_OS_SYLIXOS // Adding screens here will result in an error in order
              mWaylandIntegration->screenAdded(screen);
      #endif
          ...
      
      file: qt5\qtwayland\src\client\qwaylanddisplay.cpp
      
      Maybe add screens here:
      QWaylandDisplay::QWaylandDisplay(QWaylandIntegration *waylandIntegration)
          : mWaylandIntegration(waylandIntegration)
      {
          ...
          struct ::wl_registry *registry = wl_display_get_registry(mDisplay);
          init(registry);
      
          mWindowManagerIntegration.reset(new QWaylandWindowManagerIntegration(this));
      
          forceRoundTrip();
      
      #ifdef Q_OS_SYLIXOS
          foreach (QWaylandScreen *screen, mScreens) {
              mWaylandIntegration->screenAdded(screen);
          }
      #endif
      }
      

      Attachments

        Issue Links

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

          Activity

            People

              johanhelsing Johan Helsing
              xiaoyan xiao yan
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes