Comment 4 for bug 1518331

Revision history for this message
Jim Hodapp (jhodapp) wrote :

@Bosko: thanks for the patch. It looks pretty decent. One comment I would have immediately is to please try and match the existing C++ code style present. Use the same type of spacing, bracket styling, etc. You are also welcome to branch the mediaplayer source tree and propose a merge request via Launchpad against trunk. Find the trunk branch at lp:mediaplayer-app