[Kde-pim] Review Request 120470: C++11-narrowing causes Clang to break compilation. static_cast to solve the issue

[Jan Kundrát]

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


How does it work on these funny platform like Arm with different signedness of char? Would it make sense to use a string literal with \x escapes for each UTF-8/Latin-1 byte?

- Jan Kundrát


On Oct. 3, 2014, 1:34 a.m., Helio Castro wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120470/
> -----------------------------------------------------------
> 
> (Updated Oct. 3, 2014, 1:34 a.m.)
> 
> 
> Review request for KDEPIM and Laurent Montel.
> 
> 
> Repository: kdepim-runtime
> 
> 
> Description
> -------
> 
> C++11-narrowing causes Clang to break compilation. static_cast to solve the issue
> 
> 
> Diffs
> -----
> 
>   plugins/autotests/kcalcoreserializertest.cpp a4a2fca073d889597bc18750993162c35d5bf1b2 
> 
> Diff: https://git.reviewboard.kde.org/r/120470/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Helio Castro
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/