Review Request: Extend the scope of the playground

Ryan McCoskrie ryan.mccoskrie at gmail.com
Sat Aug 18 03:10:28 UTC 2012



> On Aug. 16, 2012, 11:14 a.m., Matěj Laitl wrote:
> > The autor says he doesn't intend to see this merged, to I presume we can close this now.
> > 
> > Ryan, as Bart says, there's some unrelated cleanup in this patch, could you please submit that as a separate review? Tracks.

Done.


- Ryan


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


On Feb. 17, 2012, 2:53 a.m., Ryan McCoskrie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103999/
> -----------------------------------------------------------
> 
> (Updated Feb. 17, 2012, 2:53 a.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> This is infrastructure for future patches of mine. It's intended to (after some revising) make it relatively easy to develop code that is (mostly) in the playground directory but is (by necessity) compiled into the player binary.
> 
> Note: I'm only looking for criticism and have no intention of seeing this shipped.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt d47c28b 
>   playground/CMakeLists.txt e96aa3f 
>   playground/src/CMakeLists.txt ed740ec 
>   src/App.cpp 2f1837c 
>   src/CMakeLists.txt 4241e69 
> 
> Diff: http://git.reviewboard.kde.org/r/103999/diff/
> 
> 
> Testing
> -------
> 
> Checked that the code compiles.
> 
> 
> Thanks,
> 
> Ryan McCoskrie
> 
>

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


More information about the Amarok-devel mailing list