<URL: http://bugs.freeciv.org/Ticket/Display.html?id=39747 >

> [chrisk - Thu Oct 04 06:42:32 2007]:
> 
> 
> SVN 04 OCT 2007 V2_1
> 
> server/auth.c
> 
>        if (show_source == SST_DEFAULT) {
>          /* TRANS: After 'Auth option "user":'. Option value may have
> been inserted
>           * between these. */
>          cat_snprintf(buffer, sizeof(buffer),
>                       _(" missing from config file (using default)"));
> 
> 
> This comment is not visible in de.po. The TRANS: flag must be set in
> all comment lines preceding the command containing the _() function, I
> think

TRANS lines must be immediately preceeding the line with the
translation.  In the above example there's a line in between (the
cat_snprintf line) so it's not connected.  Multi-line TRANS comments are
allowed however.

This is something that is probably a problem elsewhere.  It'd be nice to
have a little script to make sure every TRANS comment has a _() on the
following line.

This patch should fix that particular case.

-jason

Index: server/auth.c
===================================================================
--- server/auth.c	(revision 14083)
+++ server/auth.c	(working copy)
@@ -171,9 +171,9 @@
     switch(target->source) {
      case AOS_DEFAULT:
        if (show_source == SST_DEFAULT) {
-         /* TRANS: After 'Auth option "user":'. Option value may have been inserted
-          * between these. */
          cat_snprintf(buffer, sizeof(buffer),
+		      /* TRANS: After 'Auth option "user":'. Option value
+			 may have been inserted between these. */
                       _(" missing from config file (using default)"));
        } else {
          /* TRANS: auth option originates from internal default */
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to