The proposal to merge lp:~widelands-dev/widelands/test-ngettext into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
--
Your team Widelands Developers is subscri
Review: Approve compile / regression / review / check f1-help
Fine for me.
--
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/test-ngettext.
Review: Approve
LGTM.
--
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/test-ngettext.
___
Mailing list:
Review: Resubmit
Done :)
--
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/test-ngettext.
___
Mailing list:
The proposal to merge lp:~widelands-dev/widelands/test-ngettext into
lp:widelands has been updated.
Commit Message changed to:
Revised Lua ngettext to allow only unsigned integers. Created test for lua
gettext functions.
For more details, see:
https://code.launchpad.net/~widelands-dev
Ok, sounds good to me.
--
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/test-ngettext.
___
Mailing list:
Because the units appear in a longer sentence, I am in favour of having long
unit names rather than symbols. Let's disallow the float and remove the
placeholders. I can do that in his branch.
--
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
Your team Widelands
I fully agree with GunChleoc regarding Klaus' post. In Slovak we would say
"twenty-five whole (ones) (and) six tenths of a second" and write "25,6
sekundy", where "sekundy" is the genitive case form of the word second. That's
why a special plural form is needed.
As for the merge request, I
> > Floating point numbers can also become integers
> I don't see how that is relevant, what's it in response to?
Let's say the value is currently 25.5 - then you would need to translate "%d
seconds" with the floating point translation in Slovak. If the value changes to
25, you would need to
> Floating point numbers can also become integers
I don't see how that is relevant, what's it in response to?
--
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/test-ngettext.
Floating point numbers can also become integers - how about we extend the %s
second(s) translations to the actual numbers? On the other hand, we also have a
translation memory with fuzzy matches now, so pulling similar strings from the
Suggestions tab in Transifex shouldn't take that long...
--
> Maybe we could write a wrapper that will map floating point numbers to the
> appropriate integer numbers for the languages in our list, and then use those
> integers to call ngettext?
That wouldn't work for Slovak, because floats need their own, different form.
There is nothing to map to.
>
The actual rule for Scottish Gaelic is to discard everything after the .,
because we say "x seconds point y" for x.y seconds.
Not having things correct for Slovak doesn't please me though.
Maybe we could write a wrapper that will map floating point numbers to the
appropriate integer numbers
According to the gettext manual, "a simple gettext call with a form suitable
for all values will do"[1] -- something like http://pastebin.com/raw/R2tGW9XH.
The CLDR tables[2] clearly disprove this rule being universal, though.
Flooring produces incorrect results for Slovak and I bet a lot of
An example:
"If the food supply is steady, this mine can produce iron ore in %s on average."
%s is replaced by "%d second(s)", and %d can become floating point.
The floor is what we have always had implicitly (and it also happens to be what
my particular language does), so there is no change
Review: Approve compile, regressiontest.py +/-
Having more tests is always good and the code looks OK for me.
test-ngettext klaus$ ./regression_test.py -b ./widelands
OTOH Miroslav question is valid, where or when do we need to show someting like
"You have 3.145972 Item(plurals) in you
/_widelands_dev_widelands_test_ngettext-833.
--
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/test-ngettext into lp:widelands.
___
Mailing list: https
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/test-ngettext into lp:widelands.
___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe
/+merge/291587
We missed fixing ngettext when updating eris, so I've created a test.
--
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/test-ngettext into lp:widelands.
=== added file 'test/maps/lua_testsuite.wmf/scripting/gettext.lua
19 matches
Mail list logo
