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

REG: QPolygonF::united returns errornous result

    XMLWordPrintable

Details

    Description

      Regression issue from Qt 5.1.1.

      QPolygonF:united() returns errornous result when using more complex polygons. Please test attached polygon.zip. Mouse clicking change polygons between A, B and united (green).

      Regression is caused because of qSort is changed to std::sort in QPathClipper.

      Change-Id: I324ddf6129fe8884ecea97ef47abb7c071dfb34c

      Reverting std::sort() to qSort() fixes the issue. Patch attached. But don't believe it's solution for longer term if qSort is deprecated. Please do correction to std:sort() usage. Be aware if there's other situations also that cause problems because of the qSort change.

      Attachments

        1. pathclipper_revert.patch
          2 kB
        2. polygon.zip
          1 kB
        3. polygon2.zip
          2 kB
        4. polygon3.zip
          1 kB
        5. qtbug38037_diag.patch
          1 kB

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-34487 Wrong result using "QPolygonF united ( const QPolygonF & r ) const"

          • P4: Low - Would be nice to fix, but it's not very important.
          • Reported

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-100172 QPolygonF::united same polygons error

          • Not Evaluated - Not yet evaluated/prioritized
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-3755 Regression with QPolygonF subtracted

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              smd Jan Arve
              qtcomsupport Qt Support
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes