Review Request 111635: PlaybackConfig: add option whether playback should start on track add

[Mark Kretschmann]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111635/#review36447
-----------------------------------------------------------


Getting this build issue:

/home/mark/Devel/src/amarok/src/playlist/PlaylistController.cpp:231: error: use of undeclared identifier 'StartPlay'

- Mark Kretschmann


On July 21, 2013, 9:16 p.m., Matěj Laitl wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111635/
> -----------------------------------------------------------
> 
> (Updated July 21, 2013, 9:16 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> PlaybackConfig: add option whether playback should start on track add
> 
> CHANGES:
>  * Added an option whether adding tracks to playlist should start playing.
> 
> Also changed the entry in the 2.8 Beta ChangeLog, which is a bit strange,
> but I believe the ChangeLog is most useful for differences between major
> releases and it would be really confusing if the entry wasn't changed.
> 
> BUG: 322428
> FIXED-IN: 2.8
> 
> 
> Diffs
> -----
> 
>   ChangeLog 87e4708a6330e7a612db59052a50eef053294b06 
>   src/amarokconfig.kcfg c9d202ad1212055f5877769c7add897547bb3f38 
>   src/configdialog/dialogs/PlaybackConfig.ui 2bbcba7e2b8f7fdafe98929741e9d08a069460d0 
>   src/playlist/PlaylistController.h 185b4226c171c6a1fcd5b681721e39deae2e832b 
>   src/playlist/PlaylistController.cpp 60c99e5a10459b84b9839f07aad0d1e2bfa5d259 
> 
> Diff: http://git.reviewboard.kde.org/r/111635/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Matěj Laitl
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130724/d3007cd0/attachment.html>