Differential D7401 Diff 18366 src/solid/devices/backends/iokit/iokitvolume.h

Changeset View

Standalone View

View Options

src/solid/devices/backends/iokit/iokitvolume.h

  • This file was added.
1​/*
2​ Copyright 2017 René J.V. Bertin <rjvbertin@gmail.com>
3​
4​ This library is free software; you can redistribute it and/or
5​ modify it under the terms of the GNU Lesser General Public
6​ License as published by the Free Software Foundation; either
7​ version 2.1 of the License, or (at your option) version 3, or any
8​ later version accepted by the membership of KDE e.V. (or its
9​ successor approved by the membership of KDE e.V.), which shall
10​ act as a proxy defined in Section 6 of version 3 of the license.
11​
12​ This library is distributed in the hope that it will be useful,
13​ but WITHOUT ANY WARRANTY; without even the implied warranty of
14​ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
15​ Lesser General Public License for more details.
16​
17​ You should have received a copy of the GNU Lesser General Public
18​ License along with this library. If not, see <http://www.gnu.org/licenses/>.
19​*/
20​
21​#ifndef SOLID_BACKENDS_IOKIT_VOLUME_H
22​#define SOLID_BACKENDS_IOKIT_VOLUME_H
23​
24​#include <solid/devices/ifaces/storagevolume.h>
25​#include "iokitblock.h"
26​
27​#include <DiskArbitration/DiskArbitration.h>
28​
29​namespace Solid
30​{
31​namespace Backends
32​{
33​namespace IOKit
34​{
35​class IOKitVolume : public Block, virtual public Solid::Ifaces::StorageVolume
36​{
37​ Q_OBJECT
38​ Q_INTERFACES(Solid::Ifaces::StorageVolume)
39​
40​public:
41​ IOKitVolume(IOKitDevice *device);
42​ IOKitVolume(const IOKitDevice *device);
43​ virtual ~IOKitVolume();
44​
45​ virtual bool isIgnored() const;
kfunkUnsubmitted
Done

Here and below: Missing Q_DECL_OVERRIDE again

kfunk: Here and below: Missing `Q_DECL_OVERRIDE` again
46​ virtual Solid::StorageVolume::UsageType usage() const;
kfunkUnsubmitted
Done

No virtual needed if there's already a Q_DECL_OVERRIDE or override.

kfunk: No `virtual` needed if there's already a `Q_DECL_OVERRIDE` or `override`.
rjvbbAuthorUnsubmitted
Not Done

I guess that's safe here because IOKitOpticalDisc doesn't override any of these methods?

rjvbb: I guess that's safe here because IOKitOpticalDisc doesn't override any of these methods?
47​ virtual QString fsType() const;
48​ virtual QString label() const;
49​ virtual QString uuid() const;
50​ virtual qulonglong size() const;
51​ virtual QString encryptedContainerUdi() const;
52​
53​ const QString vendor();
kfunkUnsubmitted
Done

Here and below: No const for return types on implicitly-shared types (just adds noise)

The functions can be made const though.

kfunk: Here and below: No `const` for return types on implicitly-shared types (just adds noise) The…
54​ const QString product();
55​ const QString description();
56​
57​ DADiskRef daRef() const;
58​
59​private:
60​ class Private;
61​ Private *d;
62​};
63​}
64​}
65​}
66​
67​#endif // SOLID_BACKENDS_IOKIT_VOLUME_H