Review Request 114699: Use enums instead of ints in method types

Aleix Pol Gonzalez aleixpol at kde.org
Thu Jan 16 18:20:49 UTC 2014


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

Ship it!


Seems like a good idea :)

Potentially breaks the source compatibility, but then it improves the API considerably.

- Aleix Pol Gonzalez


On Jan. 16, 2014, 1:28 p.m., Alex Merry wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114699/
> -----------------------------------------------------------
> 
> (Updated Jan. 16, 2014, 1:28 p.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kmediaplayer
> 
> 
> Description
> -------
> 
> Use enums instead of ints in method types.
> 
> This adds to the porting effort a little (but not in many places), but gives more type-safety.
> 
> 
> Diffs
> -----
> 
>   autotests/playertest.cpp 638e86fecf7eb418c802b1f46d04e864d8e161f6 
>   autotests/testplayer.h 285da38fdf886ec8710e3a0d595ca20777426f86 
>   autotests/viewtest.cpp 337ac3eef735dab7860bb6577c72fa33d3597068 
>   src/kmediaplayer/player.h 57935bae537dba355b4c0cf2306dd6c993762661 
>   src/kmediaplayer/player.cpp c0be47666094a12abc160538798efafe1e3e1fb0 
>   src/kmediaplayer/view.h c3af06b51202b29f93ca0e3509ad41b8bbb91b0a 
>   src/kmediaplayer/view.cpp 51e4c4ba5b16a4d7b90e57f78281fedb524eca54 
> 
> Diff: https://git.reviewboard.kde.org/r/114699/diff/
> 
> 
> Testing
> -------
> 
> Builds, tests run.
> 
> 
> Thanks,
> 
> Alex Merry
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140116/997c1aa2/attachment.html>


More information about the Kde-frameworks-devel mailing list