[Okular-devel] Review Request 114019: [GCI] Extend AudioPlayer so that it gives info about if something is playing at the moment or not.

Albert Astals Cid aacid at kde.org
Sat Nov 23 16:25:13 UTC 2013


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


I'd prefer if you didn't use Phonon enum for this and used our own enum, that way if we ever want to rework the internal player to not use Phonon, there won't be a need to update the API. Also a @since marker is missing for the new method.

- Albert Astals Cid


On Nov. 22, 2013, 9:51 a.m., Egor Matirov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114019/
> -----------------------------------------------------------
> 
> (Updated Nov. 22, 2013, 9:51 a.m.)
> 
> 
> Review request for Okular.
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> Extend AudioPlayer so that it gives info about if something is playing at the moment or not according to GCI task:
>  - http://www.google-melange.com/gci/task/view/google/gci2013/5789010593054720
> 
> 
> Diffs
> -----
> 
>   core/audioplayer.h 7697562 
>   core/audioplayer.cpp af59588 
>   core/audioplayer_p.h c6d43cf 
> 
> Diff: http://git.reviewboard.kde.org/r/114019/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Egor Matirov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20131123/54093e9f/attachment.html>


More information about the Okular-devel mailing list