Clean up NepomukMeta constructors a bit
authorMatěj Laitl <matej@laitl.cz>
Wed, 19 Sep 2012 13:42:42 +0000 (15:42 +0200)
committerMatěj Laitl <matej@laitl.cz>
Wed, 19 Sep 2012 13:42:42 +0000 (15:42 +0200)
commit8db3e5d4dad6130db7abf9b63be13ebef2fedc52
treed8db389a9174917df6093b0702f843d04a02b3ff
parent3284e6c9d0daba91297a47e5aaa0d8b0cbdd4784
Clean up NepomukMeta constructors a bit

Phalgun,
 * we prefer : m_field( initvalue ) initialization to { m_field =
   initvalue }, because the first calls the constructor directly,
   second calls the assignment operator.
 * initializing class (non-primitive) types to their default values is
   redundant, C++ guarantees their constructor is called. This applies
   to QString, KShaterPtr and virtually all Qt class types.
 * I wonder that KUrl m_playableUrl = NULL; even compiles, don't use
   NULL in Amarok code at all.

Also removed unnecessary crippling of NepomukTrack::discNumber() and
a comment in NepomukArtist constructor that was invalid. (or I didn't
get it).

CCMAIL: me@phalgun.in
src/core-impl/collections/nepomukcollection/meta/NepomukAlbum.cpp
src/core-impl/collections/nepomukcollection/meta/NepomukArtist.cpp
src/core-impl/collections/nepomukcollection/meta/NepomukComposer.cpp
src/core-impl/collections/nepomukcollection/meta/NepomukTrack.cpp