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