Differential D15000 Diff 40219 src/widgets/kpropertiesdialog.cpp

Changeset View

Standalone View

View Options

src/widgets/kpropertiesdialog.cpp

Context not available.
75#include <QPushButton> 75​#include <QPushButton>
76#include <QRegularExpression> 76​#include <QRegularExpression>
77#include <QStandardPaths> 77​#include <QStandardPaths>
78​#include <QStorageInfo>
78#include <QStyle> 79​#include <QStyle>
79#include <QtConcurrentRun> 80​#include <QtConcurrentRun>
80#include <QTextStream> 81​#include <QTextStream>
Context not available.
1157 grid->addWidget(sep, curRow, 0, 1, 3); 1158​ grid->addWidget(sep, curRow, 0, 1, 3);
1158 ++curRow; 1159​ ++curRow;
1159 1160​
1161​ l = new QLabel(i18n("File system:"), d->m_frame);
broulikUnsubmitted
Not Done

Captailize: File System

broulik: Captailize: File System
1162​ grid->addWidget(l, curRow, 0, Qt::AlignRight);
1163​
1164​ QStorageInfo *fileSystem = new QStorageInfo(QLatin1String("/"));
broulikUnsubmitted
Not Done

This leaks, you can probably create it on the stack:

QStorageInfo storageInfo(...);

Also, shouldn't you pass url.toLocalFile() instead of hardcoded /?
Also, move it in the isLocal check as QStorageInfo can only work on paths, not URLs.
Also, what about KMountPoint::mountType()?

broulik: This leaks, you can probably create it on the stack: ``` QStorageInfo storageInfo(...); ```…
1165​
1166​ l = new QLabel(d->m_frame);
1167​ grid->addWidget(l, curRow++, 2);
1168​ l->setTextInteractionFlags(Qt::TextSelectableByMouse | Qt::TextSelectableByKeyboard);
1169​ l->setText(QString(fileSystem->fileSystemType()));
broulikUnsubmitted
Not Done

QString::fromLatin1?

broulik: `QString::fromLatin1`?
1170​
1160 if (isLocal) { 1171​ if (isLocal) {
1161 KMountPoint::Ptr mp = KMountPoint::currentMountPoints().findByPath(url.toLocalFile()); 1172​ KMountPoint::Ptr mp = KMountPoint::currentMountPoints().findByPath(url.toLocalFile());
1162 if (mp && mp->mountPoint() != QLatin1String("/")) { 1173​ if (mp && mp->mountPoint() != QLatin1String("/")) {
Context not available.