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

Qt 5 to-do in Mac Cocoa platform plugin

    XMLWordPrintable

Details

    • Task
    • Resolution: Done
    • P2: Important
    • 5.0.0 Beta 1
    • None
    • Core: Other
    • None
    • macOS

    Description

      qtbase/src/plugins/platforms/cocoa/qmacclipboard.mm contains the following comment, presumably to indicate something that needs fixing for Qt 5.0.0:

      qmacclipboard.mm
      #if 0
                    //### Grrr, why didn't I put in a virtual int QMacPasteboardMime::count()? --Sam
                    const int numItems = c->convertFromMime(mimeType, mimeData, flavor).size();
#else
                    int numItems = 1; //this is a hack but it is much faster than allowing conversion above
                    if (c->convertorName() == QLatin1String("FileURL"))
                        numItems = mime_src->urls().count();
#endif

      If this is not going to be fixed before Qt 5.0.0 is released (and thus not for 5.x at all, due to binary-compatibility promises), this code should be either cleaned up or marked as a Qt 6 to-do.

      Attachments

        Issue Links

          resulted from

          Sub-task - The sub-task of the issue QTBUG-23524 [API] Grep the source for Qt5 todo items

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              sorvig Morten Sørvig
              jasmcdon Jason McDonald (Closed Nokia Identity. Please assign to "macadder" instead) (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes