Differential D7155 Diff 17758 Modules/devinfo/soldevice.cpp

Changeset View

Standalone View

View Options

Modules/devinfo/soldevice.cpp

Context not available.
146{ 146​{
147 return deviceSet; 147​ return deviceSet;
148} 148​}
149​
150​bool SolDevice::operator< ( const QTreeWidgetItem & other ) const
151​{
152​ const SolDevice * otherDevice = dynamic_cast<const SolDevice*>(&other);
153​ if (otherDevice != nullptr)
broulikUnsubmitted
Done

Coding style: brace on the same line, also no need for explicit nullptr check:

if (otherDevice) {
broulik: Coding style: brace on the same line, also no need for explicit `nullptr` check: ``` if…
154​ {
155​ if (deviceType() != otherDevice->deviceType())
156​ {
157​ return deviceType() < otherDevice->deviceType();
158​ }
159​ switch (deviceType())
160​ {
161​ case Solid::DeviceInterface::Processor:
162​ {
163​ const Solid::Processor *left = tiedDevice.as<const Solid::Processor>();
164​ const Solid::Processor *right = otherDevice->tiedDevice.as<const Solid::Processor>();
165​ return left->number() > right->number();
anthonyfieroniUnsubmitted
Done

left->number() < right->number() ?

anthonyfieroni: left->number() < right->number() ?
isaactAuthorUnsubmitted
Not Done

This is reversed as it's sorted in the opposite order (ascending) to the other entries (descending)

isaact: This is reversed as it's sorted in the opposite order (ascending) to the other entries…
166​ }
167​ case Solid::DeviceInterface::StorageVolume:
168​ {
169​ return other.text(0) < text(0);
anthonyfieroniUnsubmitted
Done

Reverse again ?

anthonyfieroni: Reverse again ?
isaactAuthorUnsubmitted
Not Done

These are also sorted in ascending order, so that it goes from sda1, sda2, etc.

isaact: These are also sorted in ascending order, so that it goes from sda1, sda2, etc.
170​ }
171​ default:
172​ break;
173​ }
174​ }
175​ return text(0) < other.text(0);
176​}
177​No newline at end of file
Context not available.
anthonyfieroniUnsubmitted
Done

Add a new line.

anthonyfieroni: Add a new line.