[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Review Request: Fix typos in libkunitconversion
From:       "John Layt" <johnlayt () googlemail ! com>
Date:       2009-12-07 18:20:41
Message-ID: 20091207182041.16588.34553 () localhost
[Download RAW message or body]


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2335/
-----------------------------------------------------------

Review request for kdelibs.


Summary
-------

Actually for review by kde-i18n-doc, but no option for them or to leave group blank.

Fix spelling mistakes, incorrect plurals, and incorrect capitalisation, as discussed \
on i18n mailing list.  Most currency plurals are straightforward, some have multiple \
possibilities which I've included in a comment.  I've mostly stuck with common en_US \
usage rather than the correct original language form, but where both are common or \
disputed I've gone with original, e.g. zloty/zlotys rather than zloty or zlote, but \
real/reais rather than real/reals.  Feel free to bikeshed :-)


Diffs
-----

  trunk/KDE/kdelibs/kunitconversion/currency.cpp 1059501 
  trunk/KDE/kdelibs/kunitconversion/pressure.cpp 1059501 
  trunk/KDE/kdelibs/kunitconversion/temperature.cpp 1059501 
  trunk/KDE/kdelibs/kunitconversion/timeunit.cpp 1059501 
  trunk/KDE/kdelibs/kunitconversion/velocity.cpp 1059501 

Diff: http://reviewboard.kde.org/r/2335/diff


Testing
-------

It compiles :-)  Not sure how to test a runner when not under the full desktop, under \
my setup I only get the 4.3 krunner?


Thanks,

John


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic