Improve translator comments.

Apparently some of the TRANSLATOR comments were not clear enough. This
change makes it less easy for translators to become confused.

diff --git a/src/language/expressions/helpers.c b/src/language/expressions/helpers.c
index 7c3371f5db97eb9daf3455c943014a035d178408..720d910f686dd6993c9df4d3d19fc9762af9548e 100644 (file)
--- a/src/language/expressions/helpers.c
+++ b/src/language/expressions/helpers.c
@@ -175,12 +175,13 @@ recognize_unit (struct substring name, enum date_unit *unit)
         return true;
       }
 
-  /* TRANSLATORS: Don't translate the the actual unit names `weeks', `days' etc
-       They must remain in their original English. */
   msg (SE, _("Unrecognized date unit `%.*s'.  "
-             "Valid date units are `years', `quarters', `months', "
-             "`weeks', `days', `hours', `minutes', and `seconds'."),
-       (int) ss_length (name), ss_data (name));
+             "Valid date units are `%s', `%s', `%s', "
+             "`%s', `%s', `%s', `%s', and `%s'."),
+       (int) ss_length (name), ss_data (name),
+       "years", "quarters", "months",
+       "weeks", "days", "hours", "minutes", "seconds");
+
   return false;
 }
 
@@ -328,7 +329,7 @@ recognize_method (struct substring method_name, enum date_sum_method *method)
   else
     {
       msg (SE, _("Invalid DATESUM method.  "
-                 "Valid choices are `closest' and `rollover'."));
+                 "Valid choices are `%s' and `%s'."), "closest", "rollover");
       return false;
     }
 }

diff --git a/src/output/driver.c b/src/output/driver.c
index 5c5c8648864fd0866ed1312b405c803f68294497..acc58cf999cdd7b77d533f54edc9ce17db954550 100644 (file)
--- a/src/output/driver.c
+++ b/src/output/driver.c
@@ -315,9 +315,8 @@ output_driver_create (struct string_map *options)
     device_type = SETTINGS_DEVICE_LISTING;
   else
     {
-      /* TRANSLATORS: Don't translate the words `terminal' or `listing'. */
-      error (0, 0, _("%s is not a valid device type (the choices are "
-                     "`terminal' and `listing')"), device_string);
+      error (0, 0, _("%s is not a valid device type (the choices are `%s' and `%s')"),
+                     device_string, "terminal", "listing");
       device_type = default_device_type (file_name);
     }
 

diff --git a/src/ui/gui/help-menu.c b/src/ui/gui/help-menu.c
index 9d1ccf8246297243ef4a58c210dadbf1e086f22f..cd4d751c6af4ef6c875f63fc061c1d5bbf710a51 100644 (file)
--- a/src/ui/gui/help-menu.c
+++ b/src/ui/gui/help-menu.c
@@ -71,8 +71,8 @@ about_new (GtkMenuItem *m, GtkWindow *parent)
   gtk_about_dialog_set_translator_credits 
     (
      GTK_ABOUT_DIALOG (about),
-     /* TRANSLATORS: Use this string to list the people who have helped with
-       translation to your language. */
+     /* TRANSLATORS: Do not translate this string.  Instead, put the names of the people
+       who have helped in the translation. */
      _("translator-credits")
      );
 

diff --git a/src/ui/gui/psppire-data-window.c b/src/ui/gui/psppire-data-window.c
index 295ab65dbeba5a248852e60275ec550c14e56c75..5e9691c5d0c5ade12598de794bc406f42b41374b 100644 (file)
--- a/src/ui/gui/psppire-data-window.c
+++ b/src/ui/gui/psppire-data-window.c
@@ -1334,7 +1334,6 @@ psppire_data_window_new (struct dataset *ds)
   dw = GTK_WIDGET (
     g_object_new (
       psppire_data_window_get_type (),
-      /* TRANSLATORS: This will form a filename.  Please avoid whitespace. */
       "description", _("Data Editor"),
       "dataset", ds,
       NULL));