---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108716/#review27059
---
The patch looks good now, just some last trivial things.
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108716/
---
(Updated Feb. 10, 2013, 2:21 a.m.)
Review request for Amarok.
Changes
On Feb. 9, 2013, 5:36 p.m., Matěj Laitl wrote:
src/MainWindow.cpp, line 836
http://git.reviewboard.kde.org/r/108716/diff/2/?file=113315#file113315line836
Amarok coding style says:
connect( action, SIGNAL(triggered(bool)), this,
SLOT(slotShufflePlaylist()) );
On Feb. 9, 2013, 12:06 p.m., Matěj Laitl wrote:
src/MainWindow.cpp, line 836
http://git.reviewboard.kde.org/r/108716/diff/2/?file=113315#file113315line836
Amarok coding style says:
connect( action, SIGNAL(triggered(bool)), this,
SLOT(slotShufflePlaylist()) );
On Feb. 9, 2013, 12:06 p.m., Matěj Laitl wrote:
src/MainWindow.cpp, line 836
http://git.reviewboard.kde.org/r/108716/diff/2/?file=113315#file113315line836
Amarok coding style says:
connect( action, SIGNAL(triggered(bool)), this,
SLOT(slotShufflePlaylist()) );