extragear/multimedia/amarok

Mark Kretschmann kretschmann at kde.org
Tue Oct 7 21:01:58 CEST 2008


On Tue, Oct 7, 2008 at 8:20 PM, Soren Harward <stharward at gmail.com> wrote:
> SVN commit 868947 by stharward:
>
> Major overhaul of the playlist subsystem

Soren, your new code has many coding style violations, e.g. here:
Meta::loadPlaylist(KUrl(defaultPlaylistPath()))->tracks();

We place whitespace around brackets, like this:
Meta::loadPlaylist( KUrl( defaultPlaylistPath() ) )->tracks();


Please refer to this text for the official Amarok coding style:

amarok/HACKING/intro_and_style.txt

-- 
Mark Kretschmann
Amarok Developer
www.kde.org - amarok.kde.org


More information about the Amarok-devel mailing list