Stop after

David Dubrović david.dubrovic at globalnet.hr
Sat Dec 20 17:29:01 UTC 2008


Nikolaj Hald Nielsen wrote:
> I tried applying your patch, and functionality wise I don't see
> anything major wrong with it. Interface wise, it might be nice to have
> "stop after current" exposed somewhere i the main toolbar, but I am
> not sure how this should be done.
> 
> Even though their might be bigger plans for "stop after foo" ahead, I
> don't have any issues with committing this patch for 2.0.1 if you make
> the code adhere to out coding guidelines a bit better ( info on this
> can be found in the HACKING directory ) nd rsend the patch! :-)
> 
> - Nikolaj
> 

I corrected what I've found (mostly variable names). I believe It's ok 
now. If not please tell me so I can fix it.

-- 
Gimzo
David Dubrovic
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-stopafter-v2.patch
Type: text/x-patch
Size: 4134 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/amarok/attachments/20081220/33eea880/attachment.patch>


More information about the Amarok mailing list