[Kde-graphics-devel] Review Request 122553: Reenabled text to speech using QtSpeech as an optional dependency. Based on frameworks branch.

[Frederik Gladhorn]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122553/#review75963
-----------------------------------------------------------



part.cpp
<https://git.reviewboard.kde.org/r/122553/#comment52406>

    the space changes are unrelated, but fine by me.



part.cpp
<https://git.reviewboard.kde.org/r/122553/#comment52407>

    lots more unrelates whitespace changes



ui/pageview.cpp
<https://git.reviewboard.kde.org/r/122553/#comment52408>

    useKTTSD could be renamed



ui/pageview.cpp
<https://git.reviewboard.kde.org/r/122553/#comment52409>

    This is still commented out.


- Frederik Gladhorn


On Feb. 13, 2015, 2:48 a.m., Jeremy Whiting wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122553/
> -----------------------------------------------------------
> 
> (Updated Feb. 13, 2015, 2:48 a.m.)
> 
> 
> Review request for KDE Graphics, Albert Astals Cid and Frederik Gladhorn.
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> If QtSpeech (dev branch) is built and installed this builds and works
> very well.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt b9bc1f4d557344a293907b3c02e4b0a5c1f01cb7 
>   part.cpp 395b0e41386991d5a4e0eb340d088484c07f8a31 
>   ui/pageview.cpp f1886d3e556cbd2fe28c675b3c3317633d14d188 
>   ui/tts.h cb990e97bf7980422cd41a64cfb783cd78bf1a6c 
>   ui/tts.cpp 6e39e20138ec3f946152f6246fedd5d8dc6482e3 
> 
> Diff: https://git.reviewboard.kde.org/r/122553/diff/
> 
> 
> Testing
> -------
> 
> It builds and runs well with qtspeech. I'll test building when qtspeech isn't installed tomorrow.
> 
> 
> Thanks,
> 
> Jeremy Whiting
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-graphics-devel/attachments/20150213/9c8a5328/attachment.html>