> On April 14, 2013, 1:38 p.m., Matěj Laitl wrote: > > src/dialogs/EqualizerDialog.cpp, lines 100-102 > > <http://git.reviewboard.kde.org/r/108995/diff/4/?file=138625#file138625line100> > > > > code style: no space between if and ( > > Harsh Gupta wrote: > ooppss !!! > > Matěj Laitl wrote: > I'll fix it while committin, no need to update the patch. > > Harsh Gupta wrote: > Actually I have already done that in the next patch
Oh I see, sorry, I was confused. - Matěj ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108995/#review31016 ----------------------------------------------------------- On April 15, 2013, 3:55 p.m., Harsh Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108995/ > ----------------------------------------------------------- > > (Updated April 15, 2013, 3:55 p.m.) > > > Review request for Amarok. > > > Description > ------- > > 1. Disabled Pre-amplifier in equalizer if it is not supported by phonon. > 2. Fixed top and bottom labels of first slider. Earlier band name label was > at the top and slider value label was at the bottom for first slider. > 3. Removed an extra semicolon EqualizerDialog.h . > Note : I have made an assumption that if at all preamp is present then it > will the first element of Effect Parameter list. > > > This addresses bug 301311. > https://bugs.kde.org/show_bug.cgi?id=301311 > > > Diffs > ----- > > ChangeLog a278be3 > src/EngineController.h 5de4beb > src/EngineController.cpp 28fb256 > src/dialogs/EqualizerDialog.cpp f42a033 > src/dialogs/EqualizerDialog.ui 43b0187 > > Diff: http://git.reviewboard.kde.org/r/108995/diff/ > > > Testing > ------- > > All unit test cases passed. > Note : I have tested it with gstreamer only. Xine phonon keep crashing on my > PC. > > > File Attachments > ---------------- > > Equalizer snapshot > http://git.reviewboard.kde.org/media/uploaded/files/2013/02/17/equalizer.png > > > Thanks, > > Harsh Gupta > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
