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

BC and SC breakage in current Qt 5.6.0 release contents

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • P1: Critical
    • 5.6.0 RC
    • 5.6.0 Beta
    • GUI: Text handling
    • None
    • 3f482fad3c136b1b2f485dce823985bfa0922a3e

    Description

      Qt Base change d921a9bd157b04242722ab4326c5f2ea8e88cbea broke binary and source compatibility.

      The issue has been discussed in detail in https://codereview.qt-project.org/#/c/139410/ and https://codereview.qt-project.org/#/c/138979/.

      At its core, the change makes two previously publicly accessible but undocumented constructors of the exported QTextCursor class private (and changes the signature of one of them). The constructors were using pointers to QTextCursorPrivate and QTextDocumentPrivate, produced by other public exported functions such as QTextDocument::docHandle(). In code using this interface (such as in Qt Creator) the type of the handle is not explicitly used, it really acts as an opaque handle there.

      The author of the patch argues that the handle type and the missing documentation clearly marks it as private API independent of the fact that it is exported and publicly accessible and therefore exempts it from the usual BC and SC guarantee. Several other people disagree with this view.

      No matter of the original intent of the API or whether the API is considered unfortunate or not, the change mentioned does break both source and binary compatibility. The base change is needless insofar as there is no user or depending change benefiting from it and it ill-affects an interface that has been stable and in use for years at least in Qt Creator.

      A change to revert the SC and BC breakage is here: https://codereview.qt-project.org/#/c/147496

      Attachments

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

        Activity

          People

            esabraha Eskil Abrahamsen Blomfeldt
            hjk hjk
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes