D28590: Add a QString Solid::Device::displayName, used in Fstab Device for network mounts

Stefan BrĂ¼ns noreply at phabricator.kde.org
Sun Apr 12 11:39:37 BST 2020


bruns added inline comments.

INLINE COMMENTS

> fstabdevice.cpp:63
>      }
>  
> +    if (m_storageType == StorageType::NetworkShare) {

the `if (m_displayName.isEmpty())` block belongs here

> fstabdevice.cpp:145
> +    }
> +    const auto filePath = m_storageAccess->filePath();
> +    if (!filePath.isEmpty()) {

use `FstabHandling::mountPoints(m_device).first()` instead of m_storageAccess.

> fstabdevice.h:64
>      QString m_vendor;
> +    QString m_label;
>      QString m_description;

Wrong variable name

REPOSITORY
  R245 Solid

REVISION DETAIL
  https://phabricator.kde.org/D28590

To: meven, #frameworks, bruns, sitter
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200412/fc2aae1a/attachment.html>


More information about the Kde-frameworks-devel mailing list