[Kde-hardware-devel] Review Request: Fix description for storage volumes (UDisks)

Alex Merry kde at randomguy3.me.uk
Thu Nov 18 02:46:33 CET 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5896/
-----------------------------------------------------------

Review request for Solid and Lukáš Tinkl.


Summary
-------

UDisksDevice::volumeLabel() generates a nice description for a storage volume if there is no volume label set.  However, it checks for a volume label with UDisksStorageVolume::label(), which always returns _something_, even if that is the vendor and product pair.

There are three possible fixes for this:
* check IdLabel and PartitionLabel directly in UDisksDevice::volumeLabel() - this is what this patch does
* change UDisksStorageVolume::label() to return an empty string if there is no label set - this is what the HAL backend does
* never set the description to the volume label


Diffs
-----

  /trunk/KDE/kdelibs/solid/solid/backends/udisks/udisksdevice.cpp 1198123 

Diff: http://svn.reviewboard.kde.org/r/5896/diff


Testing
-------


Thanks,

Alex

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20101118/aa3da8d1/attachment.htm 


More information about the Kde-hardware-devel mailing list