Review Request: port sonnet away from i18nc
David Faure
faure at kde.org
Mon Dec 24 09:22:41 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107412/#review23939
-----------------------------------------------------------
staging/sonnet/src/core/loader.cpp
<http://git.reviewboard.kde.org/r/107412/#comment18253>
Maybe we want to actually provide a context, now that we know it can work, rather than an empty string.
Maybe "SonnetLoader", both here and in the translate() calls further below.
staging/sonnet/src/core/loader.cpp
<http://git.reviewboard.kde.org/r/107412/#comment18252>
lots of trailing whitespaces around here
- David Faure
On Dec. 23, 2012, 5:32 p.m., Martin Tobias Holmedahl Sandsmark wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107412/
> -----------------------------------------------------------
>
> (Updated Dec. 23, 2012, 5:32 p.m.)
>
>
> Review request for kdelibs and David Faure.
>
>
> Description
> -------
>
> port sonnet away from i18nc.
>
>
> Diffs
> -----
>
> staging/sonnet/src/core/loader.cpp 887aee5
> staging/sonnet/src/core/settings.cpp 59cb593
> staging/sonnet/src/core/speller.cpp f831f55
>
> Diff: http://git.reviewboard.kde.org/r/107412/diff/
>
>
> Testing
> -------
>
> it builds.
>
>
> Thanks,
>
> Martin Tobias Holmedahl Sandsmark
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20121224/702a56d1/attachment.htm>
More information about the kde-core-devel
mailing list