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

Make qwineventnotifier_p.h (QWinEventNotifier) a public header

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: P3: Somewhat important
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Core: Event loop
    • Labels:
      None
    • Technical Risk:
      Normal

      Description

      QWinEventNotifier is an essential class if you're using native Windows Overlapped IO and need to convert it to Qt signals. However the header is marked private (QtCore/private/qwineventnotifier_p.h). Despite this, some projects that need to use native windows IO use it anyway because it's not feasible to do it any other way. For example qextserialport uses it to convert the native WaitCommEvent to a QIODevice::readyRead() signal. See here: http://code.google.com/p/qextserialport/source/browse/src/qextserialport.h#138 . The Symbian-related plugins for Qt Creator also need to use qwineventnotifier_p.h.

      I'd like QWinEventNotifier to be made public in 4.8 so people don't have to rely on private headers that may or may not be exported in their SDK, and so Qt Creator can continue to build cleanly without needing private headers.

        Attachments

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

          Activity

            People

            Assignee:
            earthdomain Earth Domain (Inactive)
            Reporter:
            tomsci Tom Sutcliffe
            Votes:
            3 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Gerrit Reviews

                There are no open Gerrit changes