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

Qt 5 to-do's in QFileSystemModel

XMLWordPrintable

    • 94519a441cf1ea77f1422c44a7ef8ec15171ad04

      The QFileSystemModel code contains three Qt 5 to-do comments:

      qtbase/src/widgets/dialogs/qfilesystemmodel.cpp
          // Note that we match the behavior of entryList and not QFileInfo on this and this
          // incompatibility won't be fixed until Qt 5 at least
          bool isDot    = (node->fileName == QLatin1String("."));
          bool isDotDot = (node->fileName == QLatin1String(".."));
          if (   (hideHidden && !(isDot || isDotDot) && node->isHidden())
              || (hideSystem && node->isSystem())
              || (hideDirs && node->isDir())
              || (hideFiles && node->isFile())
              || (hideSymlinks && node->isSymLink())
              || (hideReadable && node->isReadable())
              || (hideWritable && node->isWritable())
              || (hideExecutable && node->isExecutable())
              || (hideDot && isDot)
              || (hideDotDot && isDotDot))
              return false;
      
      qtbase/src/widgets/dialogs/qfilesystemmodel.h
          bool rmdir(const QModelIndex &index) const; // ### Qt5: should not be const
      
      qtbase/src/widgets/dialogs/qfilesystemmodel_p.h
          // ### Qt 5: resolvedSymLinks goes away
          QHash<QString, QString> resolvedSymLinks;
      

      Any of these that can be handled without breaking source-compatibility should be actioned for Qt 5.0.0. Any others should be removed or changed to Qt 6 to-do's.

        For Gerrit Dashboard: QTBUG-25088
        # Subject Branch Project Status CR V

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

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes