Re: Russian translation

2015-02-24 2:54 GMT+05:00 Jacob Nevins wrote:

> msgid "Starting now, researching %s would need %d bulbs."
>
> I think the answer to this one is just to make it a msgid_plural.
Yes, I agree.

> # FIXME: Already pluralized msgid "%d bulb" should be used as %s instead
> # of these "%d bulbs"
> msgid_plural ""
> "To reach %s you need to obtain %d other technologies first. The whole "
> "project will require %d bulbs to complete."
>
> (We've done this before, for gold. Has this caused any translators any
> actual problems?)
I cannot speak for everyone, but a proper /* TRANS: */ comment really helps
in avoiding confusing, so I think injecting link to "%d bulb" should not
confuse.

> ...however, in this case there's no choice, so we'll have to take the
> approach you suggest.
The sentence quoted have two parts, so there is one more way to split it.
Please look at this patch. Is it too ugly, or does it violate coding policy?
===================================================================
--- helpdata.c  (revision 28306)
+++ helpdata.c  (working copy)
@@ -4078,16 +4078,19 @@
                      research_total_bulbs_required(presearch, i, FALSE));
       } else if (research_invention_reachable(presearch, i)) {
         cat_snprintf(buf, bufsz,
+                     /* TRANS: Last %s points to already translated
+                               text. */
                      PL_("To reach %s you need to obtain %d other"
-                         " technology first. The whole project"
-                         " will require %d bulbs to complete.",
+                         " technology first. %s",
                          "To reach %s you need to obtain %d other"
-                         " technologies first. The whole project"
-                         " will require %d bulbs to complete.",
+                         " technologies first.",
                          research_goal_unknown_techs(presearch, i) - 1),
                      advance_name_translation(vap),
-                     research_goal_unknown_techs(presearch, i) - 1,
-                     research_goal_bulbs_required(presearch, i));
+                     PL_("The whole project will require %d bulb"
+                         " to complete.",
+                         "The whole project will require %d bulbs"
+                         " to complete.",
+                         research_goal_bulbs_required(presearch, i)));
       } else {
         CATLSTR(buf, bufsz,
                 _("You cannot research this technology."));
===================================================================

> # FIXME: There're no "?unitclass:Diplomat" sentence to translate.
>
> Not sure quite what you want here.
> There are three uses of this unqualified string:
>  - A unit type name
>  - A unit flag name
>  - A unit role name
Yes, I meant second and third cases, mostly third: "?unitrole:Diplomat" and
unmentioned "?unitrole:Settlers", or even change them to something like
"DoesDiplomatActions" and "DoesTerrainAlteration"...

In fact strings "Dimplomat", "Settlers" with also "Spy", "Bombarder",
"Marines", etc are already occupied as unit type names and they are
untouchable, their using in other contexts leads to some translation
problems.
For example, in Russian I've found a workaround for unit flag context, but
still haven't for role.

_______________________________________________
Freeciv-i18n mailing list
Freeciv-i18n-8nu/[email protected]
https://mail.gna.org/listinfo/freeciv-i18n