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

inlining QByteArray::isNull() causes BC break on static Qt builds

    XMLWordPrintable

Details

    • All
    • 3
    • 8602a224b (dev), db9d60435 (6.4), 7a850fc00 (6.5)
    • Foundation Sprint 72

    Description

      The change https://codereview.qt-project.org/c/qt/qtbase/+/419976 introduced inlining for QByteArray::isNull(). If this function is used in a library, the application fails to build on iOS, since for iOS we currently mandate static builds.

      A sample code would be:

      // application
#include <QCoreApplication>

#include "blibrary.h"

int main(int argc, char *argv[])
{
  doBLibrary();
}

      #ifndef BLIBRARY_H
#define BLIBRARY_H

bool doBLibrary();

#endif // BLIBRARY_H

      // library
#include "blibrary.h"

#include <QByteArray>

bool doBLibrary()
{
  return QByteArray().isNull();
}

      Attachments

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-127070 QT_DISABLE_DEPRECATED_UP_TO breaks ABI with static builds

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Closed
          resulted in

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-111884 FTBFS: qtbase can't be built in static mode

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed

          Suggestion - Public suggestions QTBUG-85974 Allow dynamic builds of Qt for iOS - Tech Preview

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Open
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              mmutz Marc Mutz
              vminenko Vladimir Minenko
              Vladimir Minenko Vladimir Minenko
              Alex Blasche Alex Blasche
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes