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

Qt 5 to-do in plugin framework

    XMLWordPrintable

Details

    • Task
    • Resolution: Done
    • P2: Important
    • 5.0.0 Beta 1
    • None
    • Core: Plugins
    • None
    • ec360d7ad90945273c7d96c9a159131dcbcd67c3

    Description

      src/corelib/plugin/qplugin.h contains the following Qt 5 to-do note:

      qplugin.h
      // NOTE: if you change pattern, you MUST change the pattern in
      // qlibrary.cpp as well.  changing the pattern will break all
      // backwards compatibility as well (no old plugins will be loaded).
      // QT5: should probably remove the entire pattern thing and do the section
      //      trick for all platforms. for now, keep it and fallback to scan for it.
      #  ifdef QPLUGIN_DEBUG_STR
      #    undef QPLUGIN_DEBUG_STR
      #  endif
      #  ifdef QT_NO_DEBUG
      #    define QPLUGIN_DEBUG_STR "false"
      #    define QPLUGIN_SECTION_DEBUG_STR ""
      #  else
      #    define QPLUGIN_DEBUG_STR "true"
      #    define QPLUGIN_SECTION_DEBUG_STR ".debug"
      #  endif
      #  define Q_PLUGIN_VERIFICATION_DATA \
          static const char qt_plugin_verification_data[] = \
            "pattern=" "QT_PLUGIN_VERIFICATION_DATA" "\n" \
            "version=" QT_VERSION_STR "\n" \
            "debug=" QPLUGIN_DEBUG_STR;
      ...
      

      If this can be resolved without breaking source-compatibility, it should be done for Qt 5.0.0. If not, the comment should be updated to Qt 6 or removed.

      Attachments

        Issue Links

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

          Activity

            People

              laknoll Lars Knoll
              jasmcdon Jason McDonald (Closed Nokia Identity. Please assign to "macadder" instead) (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes