Remove redundant initialization

This commit is contained in:
Jonas Kvinge 2020-06-15 00:11:52 +02:00
parent 13b60351a6
commit 56caab4461
21 changed files with 32 additions and 38 deletions

View File

@ -86,9 +86,9 @@ class APE::Tag::TagPrivate {
// public methods // public methods
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
APE::Tag::Tag() : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) {} APE::Tag::Tag() : d(new TagPrivate()) {}
APE::Tag::Tag(Strawberry_TagLib::TagLib::File *file, long footerLocation) : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) { APE::Tag::Tag(Strawberry_TagLib::TagLib::File *file, long footerLocation) : d(new TagPrivate()) {
d->file = file; d->file = file;
d->footerLocation = footerLocation; d->footerLocation = footerLocation;

View File

@ -38,7 +38,7 @@ class ASF::Tag::TagPrivate {
AttributeListMap attributeListMap; AttributeListMap attributeListMap;
}; };
ASF::Tag::Tag() : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) { ASF::Tag::Tag() : d(new TagPrivate()) {
} }
ASF::Tag::~Tag() { ASF::Tag::~Tag() {

View File

@ -39,7 +39,7 @@ class DSDIFF::DIIN::Tag::TagPrivate {
String artist; String artist;
}; };
DSDIFF::DIIN::Tag::Tag() : Strawberry_TagLib::TagLib::Tag() { DSDIFF::DIIN::Tag::Tag() {
d = new TagPrivate; d = new TagPrivate;
} }

View File

@ -274,7 +274,7 @@ File *createInternal(FileName fileName, bool readAudioProperties, AudioPropertie
class FileRef::FileRefPrivate : public RefCounter { class FileRef::FileRefPrivate : public RefCounter {
public: public:
FileRefPrivate() : RefCounter(), file(nullptr), stream(nullptr) {} FileRefPrivate() : file(nullptr), stream(nullptr) {}
~FileRefPrivate() { ~FileRefPrivate() {
delete file; delete file;

View File

@ -35,7 +35,7 @@ using namespace IT;
class IT::File::FilePrivate { class IT::File::FilePrivate {
public: public:
explicit FilePrivate(AudioProperties::ReadStyle propertiesStyle) : tag(), properties(propertiesStyle) {} explicit FilePrivate(AudioProperties::ReadStyle propertiesStyle) : properties(propertiesStyle) {}
Mod::Tag tag; Mod::Tag tag;
IT::Properties properties; IT::Properties properties;

View File

@ -40,7 +40,7 @@ class Mod::Tag::TagPrivate {
String trackerName; String trackerName;
}; };
Mod::Tag::Tag() : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) {} Mod::Tag::Tag() : d(new TagPrivate()) {}
Mod::Tag::~Tag() { Mod::Tag::~Tag() {
delete d; delete d;

View File

@ -32,7 +32,7 @@ using namespace Strawberry_TagLib::TagLib;
class MP4::CoverArt::CoverArtPrivate : public RefCounter { class MP4::CoverArt::CoverArtPrivate : public RefCounter {
public: public:
CoverArtPrivate() : RefCounter(), format(MP4::CoverArt::JPEG) {} CoverArtPrivate() : format(MP4::CoverArt::JPEG) {}
Format format; Format format;
ByteVector data; ByteVector data;

View File

@ -32,7 +32,7 @@ using namespace Strawberry_TagLib::TagLib;
class MP4::Item::ItemPrivate : public RefCounter { class MP4::Item::ItemPrivate : public RefCounter {
public: public:
ItemPrivate() : RefCounter(), valid(true), atomDataType(TypeUndefined) {} ItemPrivate() : valid(true), atomDataType(TypeUndefined) {}
bool valid; bool valid;
AtomDataType atomDataType; AtomDataType atomDataType;

View File

@ -80,9 +80,9 @@ ByteVector ID3v1::StringHandler::render(const String &s) const {
// public methods // public methods
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
ID3v1::Tag::Tag() : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) {} ID3v1::Tag::Tag() : d(new TagPrivate()) {}
ID3v1::Tag::Tag(File *file, long tagOffset) : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) { ID3v1::Tag::Tag(File *file, long tagOffset) : d(new TagPrivate()) {
d->file = file; d->file = file;
d->tagOffset = tagOffset; d->tagOffset = tagOffset;

View File

@ -99,11 +99,11 @@ String Latin1StringHandler::parse(const ByteVector &data) const {
// public members // public members
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
ID3v2::Tag::Tag() : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) { ID3v2::Tag::Tag() : d(new TagPrivate()) {
d->factory = FrameFactory::instance(); d->factory = FrameFactory::instance();
} }
ID3v2::Tag::Tag(File *file, long tagOffset, const FrameFactory *factory) : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) { ID3v2::Tag::Tag(File *file, long tagOffset, const FrameFactory *factory) : d(new TagPrivate()) {
d->factory = factory; d->factory = factory;
d->file = file; d->file = file;
d->tagOffset = tagOffset; d->tagOffset = tagOffset;

View File

@ -57,9 +57,9 @@ class Ogg::XiphComment::XiphCommentPrivate {
// public members // public members
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
Ogg::XiphComment::XiphComment() : Strawberry_TagLib::TagLib::Tag(), d(new XiphCommentPrivate()) {} Ogg::XiphComment::XiphComment() : d(new XiphCommentPrivate()) {}
Ogg::XiphComment::XiphComment(const ByteVector &data) : Strawberry_TagLib::TagLib::Tag(), d(new XiphCommentPrivate()) { Ogg::XiphComment::XiphComment(const ByteVector &data) : d(new XiphCommentPrivate()) {
parse(data); parse(data);
} }

View File

@ -62,11 +62,11 @@ ByteVector RIFF::Info::StringHandler::render(const String &s) const {
// public members // public members
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
RIFF::Info::Tag::Tag(const ByteVector &data) : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) { RIFF::Info::Tag::Tag(const ByteVector &data) : d(new TagPrivate()) {
parse(data); parse(data);
} }
RIFF::Info::Tag::Tag() : Strawberry_TagLib::TagLib::Tag(), d(new TagPrivate()) {} RIFF::Info::Tag::Tag() : d(new TagPrivate()) {}
RIFF::Info::Tag::~Tag() { RIFF::Info::Tag::~Tag() {
delete d; delete d;

View File

@ -58,7 +58,7 @@ ByteVectorList ByteVectorList::split(const ByteVector &v, const ByteVector &patt
// public members // public members
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
ByteVectorList::ByteVectorList() : List<ByteVector>(), d(nullptr) {} ByteVectorList::ByteVectorList() : d(nullptr) {}
ByteVectorList::~ByteVectorList() {} ByteVectorList::~ByteVectorList() {}

View File

@ -28,7 +28,7 @@
using namespace Strawberry_TagLib::TagLib; using namespace Strawberry_TagLib::TagLib;
PropertyMap::PropertyMap() : SimplePropertyMap() {} PropertyMap::PropertyMap() {}
PropertyMap::PropertyMap(const SimplePropertyMap &m) { PropertyMap::PropertyMap(const SimplePropertyMap &m) {

View File

@ -145,7 +145,7 @@ namespace TagLib {
class String::StringPrivate : public RefCounter { class String::StringPrivate : public RefCounter {
public: public:
StringPrivate() : RefCounter() {} StringPrivate() {}
/*! /*!
* Stores string in UTF-16. The byte order depends on the CPU endian. * Stores string in UTF-16. The byte order depends on the CPU endian.

View File

@ -54,13 +54,13 @@ StringList StringList::split(const String &s, const String &pattern) {
// public members // public members
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
StringList::StringList() : List<String>(), d(nullptr) {} StringList::StringList() : d(nullptr) {}
StringList::StringList(const String &s) : List<String>(), d(nullptr) { StringList::StringList(const String &s) : d(nullptr) {
append(s); append(s);
} }
StringList::StringList(const ByteVectorList &bl, String::Type t) : List<String>() { StringList::StringList(const ByteVectorList &bl, String::Type t) {
ByteVectorList::ConstIterator i = bl.begin(); ByteVectorList::ConstIterator i = bl.begin();
for (; i != bl.end(); i++) { for (; i != bl.end(); i++) {

View File

@ -82,8 +82,7 @@ class Reader {
class SkipReader : public Reader { class SkipReader : public Reader {
public: public:
explicit SkipReader(unsigned int size) : m_size(size) { explicit SkipReader(unsigned int size) : m_size(size) {}
}
unsigned int read(Strawberry_TagLib::TagLib::File &file, unsigned int limit) { unsigned int read(Strawberry_TagLib::TagLib::File &file, unsigned int limit) {
unsigned int count = std::min(m_size, limit); unsigned int count = std::min(m_size, limit);
@ -102,8 +101,7 @@ class SkipReader : public Reader {
template<typename T> template<typename T>
class ValueReader : public Reader { class ValueReader : public Reader {
public: public:
explicit ValueReader(T &_value) : value(_value) { explicit ValueReader(T &_value) : value(_value) {}
}
protected: protected:
T &value; T &value;
@ -306,9 +304,7 @@ class StructReader : public Reader {
class XM::File::FilePrivate { class XM::File::FilePrivate {
public: public:
explicit FilePrivate(AudioProperties::ReadStyle propertiesStyle) explicit FilePrivate(AudioProperties::ReadStyle propertiesStyle) : properties(propertiesStyle) {}
: tag(), properties(propertiesStyle) {
}
Mod::Tag tag; Mod::Tag tag;
XM::Properties properties; XM::Properties properties;

View File

@ -97,8 +97,8 @@ class DebugBase : public QDebug {
// Debug message will be stored in a buffer. // Debug message will be stored in a buffer.
class BufferedDebug : public DebugBase<BufferedDebug> { class BufferedDebug : public DebugBase<BufferedDebug> {
public: public:
BufferedDebug() : DebugBase() {} BufferedDebug() {}
BufferedDebug(QtMsgType) : DebugBase(), buf_(new QBuffer, later_deleter) { BufferedDebug(QtMsgType) : buf_(new QBuffer, later_deleter) {
buf_->open(QIODevice::WriteOnly); buf_->open(QIODevice::WriteOnly);
// QDebug doesn't have a method to set a new io device, but swap() allows the devices to be swapped between two instances. // QDebug doesn't have a method to set a new io device, but swap() allows the devices to be swapped between two instances.
@ -116,7 +116,7 @@ class BufferedDebug : public DebugBase<BufferedDebug> {
// Debug message will be logged immediately. // Debug message will be logged immediately.
class LoggedDebug : public DebugBase<LoggedDebug> { class LoggedDebug : public DebugBase<LoggedDebug> {
public: public:
LoggedDebug() : DebugBase() {} LoggedDebug() {}
LoggedDebug(QtMsgType t) : DebugBase(t) { nospace() << kMessageHandlerMagic; } LoggedDebug(QtMsgType t) : DebugBase(t) { nospace() << kMessageHandlerMagic; }
}; };

View File

@ -72,8 +72,7 @@ const char *GstEngine::kDirectSoundSink = "directsoundsink";
const char *GstEngine::kOSXAudioSink = "osxaudiosink"; const char *GstEngine::kOSXAudioSink = "osxaudiosink";
GstEngine::GstEngine(TaskManager *task_manager) GstEngine::GstEngine(TaskManager *task_manager)
: Engine::Base(), : task_manager_(task_manager),
task_manager_(task_manager),
buffering_task_id_(-1), buffering_task_id_(-1),
latest_buffer_(nullptr), latest_buffer_(nullptr),
stereo_balancer_enabled_(false), stereo_balancer_enabled_(false),

View File

@ -39,8 +39,7 @@
#include "vlcscopedref.h" #include "vlcscopedref.h"
VLCEngine::VLCEngine(TaskManager *task_manager) VLCEngine::VLCEngine(TaskManager *task_manager)
: EngineBase(), : instance_(nullptr),
instance_(nullptr),
player_(nullptr), player_(nullptr),
state_(Engine::Empty) { state_(Engine::Empty) {

View File

@ -363,7 +363,7 @@ QWidget *TextItemDelegate::createEditor(QWidget *parent, const QStyleOptionViewI
return new QLineEdit(parent); return new QLineEdit(parent);
} }
TagCompletionModel::TagCompletionModel(CollectionBackend *backend, Playlist::Column column) : QStringListModel() { TagCompletionModel::TagCompletionModel(CollectionBackend *backend, Playlist::Column column) {
QString col = database_column(column); QString col = database_column(column);
if (!col.isEmpty()) { if (!col.isEmpty()) {