Git commit 89b07a7a747c9ce196e08317e4b3c3dff4cee52b by Casian Andrei. Committed on 18/01/2014 at 18:13. Pushed by casianandrei into branch 'five'. Remove the index attribute from the object descriptions M +10 -21 phonon/objectdescription.cpp M +4 -5 phonon/objectdescription.h M +3 -8 phonon/objectdescription_p.h http://commits.kde.org/phonon/89b07a7a747c9ce196e08317e4b3c3dff4cee52b diff --git a/phonon/objectdescription.cpp b/phonon/objectdescription.cpp index ba13c8f..9f28850 100644 --- a/phonon/objectdescription.cpp +++ b/phonon/objectdescription.cpp @@ -37,18 +37,16 @@ class DescriptionBasePrivate public: DescriptionBasePrivate() : valid(false) - , index(-1) , name() , description() { qDebug() << Q_FUNC_INFO; } = - void init(int _index, QString _name, QString _description) + void init(QString _name, QString _description) { - qDebug() << Q_FUNC_INFO << _index << _name << _description; + qDebug() << Q_FUNC_INFO << _name << _description; valid =3D true; - index =3D _index; name =3D _name; description =3D _description; } @@ -57,7 +55,6 @@ public: { qDebug() << Q_FUNC_INFO; return (valid =3D=3D other.valid) && - (index =3D=3D other.index) && (name =3D=3D other.name) && (description =3D=3D other.description); } @@ -66,14 +63,12 @@ public: { qDebug() << Q_FUNC_INFO; return (valid !=3D other.valid) && - (index !=3D other.index) && (name !=3D other.name) && (description !=3D other.description); } = bool valid; = - int index; QString name; QString description; = @@ -87,12 +82,6 @@ DescriptionBase::DescriptionBase() qDebug() << Q_FUNC_INFO; } = -int DescriptionBase::index() const -{ - P_D(const DescriptionBase); - return d->index; -} - QString DescriptionBase::name() const { P_D(const DescriptionBase); @@ -153,12 +142,12 @@ EffectDescription::EffectDescription() qDebug() << Q_FUNC_INFO; } = -EffectDescription::EffectDescription(int index, QString name, QString desc= ription) +EffectDescription::EffectDescription(QString name, QString description) : DescriptionBase() { qDebug() << Q_FUNC_INFO; P_D(DescriptionBase); - d->init(index, name, description); + d->init(name, description); } = class DeviceDescriptionBasePrivate : public DescriptionBasePrivate @@ -211,12 +200,12 @@ AudioOutputDevice::AudioOutputDevice() qDebug() << Q_FUNC_INFO; } = -AudioOutputDevice::AudioOutputDevice(int index, QString name, QString desc= ription) +AudioOutputDevice::AudioOutputDevice(QString name, QString description) : DeviceDescriptionBase() { qDebug() << Q_FUNC_INFO; P_D(DeviceDescriptionBase); - d->init(index, name, description); + d->init(name, description); } = AudioOutputDevice::~AudioOutputDevice() @@ -230,12 +219,12 @@ AudioCaptureDevice::AudioCaptureDevice() qDebug() << Q_FUNC_INFO; } = -AudioCaptureDevice::AudioCaptureDevice(int index, QString name, QString de= scription) +AudioCaptureDevice::AudioCaptureDevice(QString name, QString description) : DeviceDescriptionBase() { qDebug() << Q_FUNC_INFO; P_D(DeviceDescriptionBase); - d->init(index, name, description); + d->init(name, description); } = AudioCaptureDevice::~AudioCaptureDevice() @@ -249,12 +238,12 @@ VideoCaptureDevice::VideoCaptureDevice() qDebug() << Q_FUNC_INFO; } = -VideoCaptureDevice::VideoCaptureDevice(int index, QString name, QString de= scription) +VideoCaptureDevice::VideoCaptureDevice(QString name, QString description) : DeviceDescriptionBase() { qDebug() << Q_FUNC_INFO; P_D(DeviceDescriptionBase); - d->init(index, name, description); + d->init(name, description); } = VideoCaptureDevice::~VideoCaptureDevice() diff --git a/phonon/objectdescription.h b/phonon/objectdescription.h index cdf0608..7c7cc6a 100644 --- a/phonon/objectdescription.h +++ b/phonon/objectdescription.h @@ -42,7 +42,6 @@ class DescriptionBasePrivate; class PHONON_EXPORT DescriptionBase { public: - int index() const; QString name() const; QString description() const; #warning need QString iconHint name of icon to use for this device @@ -94,7 +93,7 @@ class PHONON_EXPORT AudioOutputDevice : public DeviceDesc= riptionBase public: AudioOutputDevice(); #warning possibly needs available param - AudioOutputDevice(int index, QString name, QString description); + AudioOutputDevice(QString name, QString description); ~AudioOutputDevice(); }; = @@ -102,7 +101,7 @@ class PHONON_EXPORT AudioCaptureDevice : public DeviceD= escriptionBase { public: AudioCaptureDevice(); - AudioCaptureDevice(int index, QString name, QString description); + AudioCaptureDevice(QString name, QString description); ~AudioCaptureDevice(); }; = @@ -110,7 +109,7 @@ class PHONON_EXPORT VideoCaptureDevice : public DeviceD= escriptionBase { public: VideoCaptureDevice(); - VideoCaptureDevice(int index, QString name, QString description); + VideoCaptureDevice(QString name, QString description); ~VideoCaptureDevice(); }; = @@ -118,7 +117,7 @@ class PHONON_EXPORT EffectDescription : public Descript= ionBase { public: EffectDescription(); /* constructs invalid description */ - EffectDescription(int index, QString name, QString description); + EffectDescription(QString name, QString description); }; = = diff --git a/phonon/objectdescription_p.h b/phonon/objectdescription_p.h index d5aa21d..ffe01df 100644 --- a/phonon/objectdescription_p.h +++ b/phonon/objectdescription_p.h @@ -34,9 +34,8 @@ namespace Phonon class ObjectDescriptionPrivate { public: - ObjectDescriptionPrivate(int _index, const QHash &_properties) - : index(_index), - name(_properties["name"].toString()), + ObjectDescriptionPrivate(const QHash &_p= roperties) + : name(_properties["name"].toString()), description(_properties["description"].toString()), properties(_properties) { @@ -44,13 +43,9 @@ namespace Phonon = bool operator=3D=3D(const ObjectDescriptionPrivate &rhs) const { - if (index =3D=3D rhs.index && (name !=3D rhs.name || descr= iption !=3D rhs.description)) - pError() << "Same index (" << index << - "), but different name/description. This is a bug = in the Phonon backend."; - return index =3D=3D rhs.index;// && name =3D=3D rhs.name &= & description =3D=3D rhs.description; + return name =3D=3D rhs.name && description =3D=3D rhs.desc= ription; } = - int index; QString name, description; QHash properties; };