Uploaded image for project: 'Qbs ("Cubes")'
  1. Qbs ("Cubes")
  2. QBS-556

Typo in XcodeProbe::compareVersions()

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • P2: Important
    • 1.2.1
    • 1.1.1, 1.2.0
    • General
    • None
    • 95ed35296e811d5ed2583e9ec84f1c26ce021a39

    Description

      src/app/qbs-setup-toolchains/xcodeprobe.cpp 86

      int XcodeProbe::compareVersions(const QString &v1, const QString &v2)
{
    QStringList v1L = v1.split(QLatin1Char('.'));
    QStringList v2L = v2.split(QLatin1Char('.'));
    int i = 0;
    while (v1.length() > i && v1.length() > i) { // <---
      ...
    }

}

      I think should be "while (v1.length() > i && v2.length() > i)"

      Attachments

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

        Activity

          People

            jbornema Joerg Bornemann
            maksqwe Maks Naumov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes