]> git.ipfire.org Git - thirdparty/gettext.git/commitdiff
Support Python brace format.
authorDaiki Ueno <ueno@gnu.org>
Mon, 22 Apr 2013 11:01:12 +0000 (20:01 +0900)
committerDaiki Ueno <ueno@gnu.org>
Fri, 26 Apr 2013 09:54:39 +0000 (18:54 +0900)
20 files changed:
gettext-tools/ChangeLog
gettext-tools/doc/ChangeLog
gettext-tools/doc/gettext.texi
gettext-tools/libgettextpo/ChangeLog
gettext-tools/libgettextpo/Makefile.am
gettext-tools/src/ChangeLog
gettext-tools/src/FILES
gettext-tools/src/Makefile.am
gettext-tools/src/format-python-brace.c [new file with mode: 0644]
gettext-tools/src/format.c
gettext-tools/src/format.h
gettext-tools/src/message.c
gettext-tools/src/message.h
gettext-tools/src/x-python.h
gettext-tools/src/xgettext.c
gettext-tools/tests/ChangeLog
gettext-tools/tests/Makefile.am
gettext-tools/tests/format-python-brace-1 [new file with mode: 0755]
gettext-tools/tests/format-python-brace-2 [new file with mode: 0755]
gettext-tools/woe32dll/gettextsrc-exports.c

index ca9e2f0f30dc516ed7a2b7efb940200fac8d7400..76cfa04482c0402edf25b8aa5f2e6368962db41f 100644 (file)
@@ -1,3 +1,8 @@
+2013-04-26  Daiki Ueno  <ueno@gnu.org>
+
+       Support for Python brace format.
+       * woe32dll/gettextsrc-exports.c: Export formatstring_python_brace.
+
 2013-04-17  Andreas Stricker  <astricker@futurelab.ch>
 
        Support for JavaScript.
index 6eccb0655fe605767cb16d2823b2727088fd9a68..5cf3278b94bf86ee8c38f4c1cd60968c179f71c8 100644 (file)
@@ -1,3 +1,9 @@
+2013-04-26  Daiki Ueno  <ueno@gnu.org>
+
+       Support for Python brace format.
+       * gettext.texi (PO Files): Mention python-brace-format.
+       (python-format): Mention brace format.
+
 2013-04-17  Andreas Stricker  <astricker@futurelab.ch>
 
        Support for JavaScript.
index e09632c9d4edd5a645b9a3fbe43b235a03062f2b..2b4ee666de7a88852eea5a899b7a94e486ec8a92 100644 (file)
@@ -1507,6 +1507,12 @@ Likewise for Shell, see @ref{sh-format}.
 @kwindex no-python-format@r{ flag}
 Likewise for Python, see @ref{python-format}.
 
+@item python-brace-format
+@kwindex python-brace-format@r{ flag}
+@itemx no-python-brace-format
+@kwindex no-python-brace-format@r{ flag}
+Likewise for Python brace, see @ref{python-format}.
+
 @item lisp-format
 @kwindex lisp-format@r{ flag}
 @itemx no-lisp-format
@@ -9059,7 +9065,12 @@ a variable reference is ignored.
 @node python-format, lisp-format, sh-format, Translators for other Languages
 @subsection Python Format Strings
 
-Python format strings are described in
+There are two kinds of format strings in Python: those acceptable to
+the Python built-in format operator @code{%}, labelled as
+@samp{python-format}, and those acceptable to the @code{format} method
+of the @samp{str} object.
+
+Python @code{%} format strings are described in
 @w{Python Library reference} /
 @w{2. Built-in Types, Exceptions and Functions} /
 @w{2.2. Built-in Types} /
@@ -9067,6 +9078,9 @@ Python format strings are described in
 @w{2.2.6.2. String Formatting Operations}.
 @uref{http://www.python.org/doc/2.2.1/lib/typesseq-strings.html}.
 
+Python brace format strings are described in @w{PEP 3101 -- Advanced
+String Formatting}, @uref{http://www.python.org/dev/peps/pep-3101/}.
+
 @node lisp-format, elisp-format, python-format, Translators for other Languages
 @subsection Lisp Format Strings
 
index 633f84e86838941f12e739f23e4c070c90085c04..99f883b4719941919af2c52581d1eae8515d8799 100644 (file)
@@ -1,3 +1,8 @@
+2013-04-26  Daiki Ueno  <ueno@gnu.org>
+
+       Support for Python brace format.
+       * Makefile.am (libgettextpo_la_AUXSOURCES): Add format-python-brace.c.
+
 2013-04-17  Andreas Stricker  <astricker@futurelab.ch>
 
        Support for JavaScript.
index 417f44a50f40e1e5ea38588edfe7bff61b2d2e21..69f0bfeca239669d457ff5f7b099f1fef7c347cf 100644 (file)
@@ -65,6 +65,7 @@ libgettextpo_la_AUXSOURCES = \
   ../src/format-c.c \
   ../src/format-sh.c \
   ../src/format-python.c \
+  ../src/format-python-brace.c \
   ../src/format-lisp.c \
   ../src/format-elisp.c \
   ../src/format-librep.c \
index ff1a392bbfea0bf96f15e9b2935c695e054ec124..5861f735a55232d42170db4928ecbfb1dc6a4dda 100644 (file)
@@ -1,3 +1,17 @@
+2013-04-26  Daiki Ueno  <ueno@gnu.org>
+
+       Support for Python brace format.
+       * message.h (format_type): New enum value 'format_python_brace.
+       (NFORMATS): Increment.
+       * message.c (format_language): Add format_python_brace entry.
+       (format_language_pretty): Likewise.
+       * format.h (formatstring_python_brace): New declaration.
+       * format-python-brace.c: New file.
+       * format.c (formatstring_parsers): Add formatstring_python_brace.
+       * x-python.h (SCANNERS_PYTHON): Refar to formatstring_python_brace.
+       * xgettext.c (xgettext_record_flag): Handle format_python_brace.
+       * FILES: Update.
+
 2013-04-22  Daiki Ueno  <ueno@gnu.org>
 
        Make msgfmt --check-header more reliable.
index 02bf07966120a1bf605f57b7523def4ae46167e1..94d7f6482a567cdf7ee93c17f1d4c427c1ae3b8f 100644 (file)
@@ -214,6 +214,7 @@ format-c.c             Format string handling for C.
 format-c-parse.h         Format string handling for C, parsing routine.
 format-sh.c            Format string handling for Shell.
 format-python.c        Format string handling for Python.
+format-python-brace.c  Format string handling for Python, braced syntax.
 format-lisp.c          Format string handling for Common Lisp.
 format-elisp.c         Format string handling for Emacs Lisp.
 format-librep.c        Format string handling for librep.
index ccd7d24d8cec88ce01965544a48ce49fa676e4ba..d5fe699b5b7b1b8e4bbef65cab70a6fd92f25211 100644 (file)
@@ -118,6 +118,7 @@ FORMAT_SOURCE += \
   format-c.c format-c-parse.h \
   format-sh.c \
   format-python.c \
+  format-python-brace.c \
   format-lisp.c \
   format-elisp.c \
   format-librep.c \
diff --git a/gettext-tools/src/format-python-brace.c b/gettext-tools/src/format-python-brace.c
new file mode 100644 (file)
index 0000000..2081536
--- /dev/null
@@ -0,0 +1,482 @@
+/* Python brace format strings.
+   Copyright (C) 2004, 2006-2007, 2013 Free Software Foundation, Inc.
+   Written by Daiki Ueno <ueno@gnu.org>, 2013.
+
+   This program is free software: you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+#ifdef HAVE_CONFIG_H
+# include <config.h>
+#endif
+
+#include <stdbool.h>
+#include <stdlib.h>
+#include <string.h>
+
+#include "format.h"
+#include "xalloc.h"
+#include "xvasprintf.h"
+#include "format-invalid.h"
+#include "gettext.h"
+
+#define _(str) gettext (str)
+
+/* Python brace format strings are defined by PEP3101 together with
+   'format' method of string class.
+   A format string directive here consists of
+     - an opening brace '{',
+     - an identifier [_A-Za-z][_0-9A-Za-z]*|[0-9]+,
+     - an optional getattr ('.') or getitem ('['..']') operator with
+       an identifier as argument,
+     - an optional width specifier starting with ':', with a
+       (unnested) format string as argument,
+     - a closing brace '}'.
+   Brace characters '{' and '}' can be escaped by doubles '{{' and '}}'.
+ */
+
+struct named_arg
+{
+  char *name;
+};
+
+struct spec
+{
+  unsigned int directives;
+  unsigned int named_arg_count;
+  unsigned int allocated;
+  struct named_arg *named;
+};
+
+
+static bool parse_upto (struct spec *spec, const char **formatp,
+                        bool is_toplevel, char terminator,
+                        bool translated, char *fdi, char **invalid_reason);
+static void free_named_args (struct spec *spec);
+
+
+/* All the parse_* functions (except parse_upto) follow the same
+   calling convention.  FORMATP shall point to the beginning of a token.
+   If parsing succeeds, FORMATP will point to the next character after
+   the token, and true is returned.  Otherwise, FORMATP will be
+   unchanged and false is returned.  */
+
+static bool
+parse_named_field (struct spec *spec,
+                   const char **formatp, bool translated, char *fdi,
+                   char **invalid_reason)
+{
+  const char *format = *formatp;
+  char c;
+
+  c = *format;
+  if ((c >= 'A' && c <= 'Z') || (c >= 'a' && c <= 'z') || c == '_')
+    {
+      do
+        c = *++format;
+      while ((c >= 'A' && c <= 'Z') || (c >= 'a' && c <= 'z') || c == '_'
+             || (c >= '0' && c <= '9'));
+      *formatp = format;
+      return true;
+    }
+  return false;
+}
+
+static bool
+parse_numeric_field (struct spec *spec,
+                     const char **formatp, bool translated, char *fdi,
+                     char **invalid_reason)
+{
+  const char *format = *formatp;
+  char c;
+
+  c = *format;
+  if (c >= '0' && c <= '9')
+    {
+      do
+        c = *++format;
+      while (c >= '0' && c <= '9');
+      *formatp = format;
+      return true;
+    }
+  return false;
+}
+
+static bool
+parse_directive (struct spec *spec,
+                 const char **formatp, bool is_toplevel,
+                 bool translated, char *fdi, char **invalid_reason)
+{
+  const char *format = *formatp;
+  const char *const format_start = format;
+  const char *name_start;
+  char c;
+
+  c = *++format;
+  if (c == '{')
+    {
+      *formatp = ++format;
+      return true;
+    }
+
+  name_start = format;
+  if (!parse_named_field (spec, &format, translated, fdi, invalid_reason)
+      && !parse_numeric_field (spec, &format, translated, fdi, invalid_reason))
+    {
+      *invalid_reason =
+        xasprintf (_("In the directive number %u, '%c' cannot start a field name."), spec->directives, *format);
+      FDI_SET (format, FMTDIR_ERROR);
+      return false;
+    }
+
+  c = *format;
+  if (c == '.')
+    {
+      format++;
+      if (!parse_named_field (spec, &format, translated, fdi,
+                              invalid_reason))
+        {
+          *invalid_reason =
+            xasprintf (_("In the directive number %u, '%c' cannot start a getattr argument."), spec->directives, *format);
+          FDI_SET (format, FMTDIR_ERROR);
+          return false;
+        }
+      c = *format;
+    }
+  else if (c == '[')
+    {
+      format++;
+      if (!parse_named_field (spec, &format, translated, fdi,
+                              invalid_reason)
+          && !parse_numeric_field (spec, &format, translated, fdi,
+                                   invalid_reason))
+        {
+          *invalid_reason =
+            xasprintf (_("In the directive number %u, '%c' cannot start a getitem argument."), spec->directives, *format);
+          FDI_SET (format, FMTDIR_ERROR);
+          return false;
+        }
+
+      c = *format++;
+      if (c != ']')
+        {
+          *invalid_reason = INVALID_UNTERMINATED_DIRECTIVE ();
+          FDI_SET (format, FMTDIR_ERROR);
+          return false;
+        }
+      c = *format;
+    }
+
+  if (c == ':')
+    {
+      if (!is_toplevel)
+        {
+          *invalid_reason =
+            xasprintf (_("In the directive number %u, no more nesting is allowed in a format specifier."), spec->directives);
+          FDI_SET (format, FMTDIR_ERROR);
+          return false;
+        }
+
+      format++;
+      if (!parse_upto (spec, &format, false, '}', translated, fdi,
+                       invalid_reason))
+        {
+          /* FDI and INVALID_REASON will be set by a recursive call of
+             parse_directive.  */
+          return false;
+        }
+
+      if (*format == '\0')
+        {
+          *invalid_reason = INVALID_UNTERMINATED_DIRECTIVE ();
+          FDI_SET (format, FMTDIR_ERROR);
+          return false;
+        }
+      c = *format;
+    }
+
+  if (c != '}')
+    {
+      *invalid_reason =
+        xasprintf (_("In the directive number %u, there is an unterminated format directive."), spec->directives);
+      FDI_SET (format, FMTDIR_ERROR);
+      return false;
+    }
+
+  if (is_toplevel)
+    {
+      char *name;
+      size_t n = format - name_start;
+
+      FDI_SET (name_start - 1, FMTDIR_START);
+
+      name = XNMALLOC (n + 1, char);
+      memcpy (name, name_start, n);
+      name[n] = '\0';
+
+      spec->directives++;
+
+      if (spec->allocated == spec->named_arg_count)
+        {
+          spec->allocated = 2 * spec->allocated + 1;
+          spec->named = (struct named_arg *) xrealloc (spec->named, spec->allocated * sizeof (struct named_arg));
+        }
+      spec->named[spec->named_arg_count].name = name;
+      spec->named_arg_count++;
+
+      FDI_SET (format, FMTDIR_END);
+    }
+
+  *formatp = ++format;
+  return true;
+}
+
+static bool
+parse_upto (struct spec *spec,
+            const char **formatp, bool is_toplevel, char terminator,
+            bool translated, char *fdi, char **invalid_reason)
+{
+  const char *format = *formatp;
+
+  for (; *format != terminator && *format != '\0';)
+    {
+      if (*format == '{')
+        {
+          if (!parse_directive (spec, &format, is_toplevel, translated, fdi,
+                                invalid_reason))
+            return false;
+        }
+      else
+        format++;
+    }
+
+  *formatp = format;
+  return true;
+}
+
+static int
+named_arg_compare (const void *p1, const void *p2)
+{
+  return strcmp (((const struct named_arg *) p1)->name,
+                 ((const struct named_arg *) p2)->name);
+}
+
+static void *
+format_parse (const char *format, bool translated, char *fdi,
+              char **invalid_reason)
+{
+  struct spec spec;
+  struct spec *result;
+
+  spec.directives = 0;
+  spec.named_arg_count = 0;
+  spec.allocated = 0;
+  spec.named = NULL;
+
+  if (!parse_upto (&spec, &format, true, '\0', translated, fdi, invalid_reason))
+    {
+      free_named_args (&spec);
+      return NULL;
+    }
+
+  /* Sort the named argument array, and eliminate duplicates.  */
+  if (spec.named_arg_count > 1)
+    {
+      unsigned int i, j;
+
+      qsort (spec.named, spec.named_arg_count, sizeof (struct named_arg),
+             named_arg_compare);
+
+      /* Remove duplicates: Copy from i to j, keeping 0 <= j <= i.  */
+      for (i = j = 0; i < spec.named_arg_count; i++)
+        if (j > 0 && strcmp (spec.named[i].name, spec.named[j-1].name) == 0)
+          free (spec.named[i].name);
+        else
+          {
+            if (j < i)
+              spec.named[j].name = spec.named[i].name;
+            j++;
+          }
+      spec.named_arg_count = j;
+    }
+
+  result = XMALLOC (struct spec);
+  *result = spec;
+  return result;
+}
+
+static void
+free_named_args (struct spec *spec)
+{
+  if (spec->named != NULL)
+    {
+      unsigned int i;
+      for (i = 0; i < spec->named_arg_count; i++)
+        free (spec->named[i].name);
+      free (spec->named);
+    }
+}
+
+static void
+format_free (void *descr)
+{
+  struct spec *spec = (struct spec *) descr;
+
+  free_named_args (spec);
+  free (spec);
+}
+
+static int
+format_get_number_of_directives (void *descr)
+{
+  struct spec *spec = (struct spec *) descr;
+
+  return spec->directives;
+}
+
+static bool
+format_check (void *msgid_descr, void *msgstr_descr, bool equality,
+              formatstring_error_logger_t error_logger,
+              const char *pretty_msgid, const char *pretty_msgstr)
+{
+  struct spec *spec1 = (struct spec *) msgid_descr;
+  struct spec *spec2 = (struct spec *) msgstr_descr;
+  bool err = false;
+
+  if (spec1->named_arg_count + spec2->named_arg_count > 0)
+    {
+      unsigned int i, j;
+      unsigned int n1 = spec1->named_arg_count;
+      unsigned int n2 = spec2->named_arg_count;
+
+      /* Check the argument names in spec1 are contained in those of spec2.
+         Both arrays are sorted.  We search for the differences.  */
+      for (i = 0, j = 0; i < n1 || j < n2; )
+        {
+          int cmp = (i >= n1 ? 1 :
+                     j >= n2 ? -1 :
+                     strcmp (spec1->named[i].name, spec2->named[j].name));
+
+          if (cmp > 0)
+            {
+              if (equality)
+                {
+                  if (error_logger)
+                    error_logger (_("a format specification for argument '%s' doesn't exist in '%s'"),
+                                  spec2->named[i].name, pretty_msgid);
+                  err = true;
+                  break;
+                }
+              else
+                j++;
+            }
+          else if (cmp < 0)
+            {
+              if (equality)
+                {
+                  if (error_logger)
+                    error_logger (_("a format specification for argument '%s' doesn't exist in '%s'"),
+                                  spec1->named[i].name, pretty_msgstr);
+                  err = true;
+                  break;
+                }
+              else
+                i++;
+            }
+          else
+            j++, i++;
+        }
+    }
+
+  return err;
+}
+
+
+struct formatstring_parser formatstring_python_brace =
+{
+  format_parse,
+  format_free,
+  format_get_number_of_directives,
+  NULL,
+  format_check
+};
+
+
+#ifdef TEST
+
+/* Test program: Print the argument list specification returned by
+   format_parse for strings read from standard input.  */
+
+#include <stdio.h>
+
+static void
+format_print (void *descr)
+{
+  struct spec *spec = (struct spec *) descr;
+  unsigned int i;
+
+  if (spec == NULL)
+    {
+      printf ("INVALID");
+      return;
+    }
+
+  printf ("{");
+  for (i = 0; i < spec->named_arg_count; i++)
+    {
+      if (i > 0)
+        printf (", ");
+      printf ("'%s'", spec->named[i].name);
+    }
+  printf ("}");
+}
+
+int
+main ()
+{
+  for (;;)
+    {
+      char *line = NULL;
+      size_t line_size = 0;
+      int line_len;
+      char *invalid_reason;
+      void *descr;
+
+      line_len = getline (&line, &line_size, stdin);
+      if (line_len < 0)
+        break;
+      if (line_len > 0 && line[line_len - 1] == '\n')
+        line[--line_len] = '\0';
+
+      invalid_reason = NULL;
+      descr = format_parse (line, false, NULL, &invalid_reason);
+
+      format_print (descr);
+      printf ("\n");
+      if (descr == NULL)
+        printf ("%s\n", invalid_reason);
+
+      free (invalid_reason);
+      free (line);
+    }
+
+  return 0;
+}
+
+/*
+ * For Emacs M-x compile
+ * Local Variables:
+ * compile-command: "/bin/sh ../libtool --tag=CC --mode=link gcc -o a.out -static -O -g -Wall -I.. -I../gnulib-lib -I../intl -DHAVE_CONFIG_H -DTEST format-python-brace.c ../gnulib-lib/libgettextlib.la"
+ * End:
+ */
+
+#endif /* TEST */
index b52e6440b815166badf92f0634f2298516fecc2c..c73ad7de9d077e122b0e69f5a53522d3ff31d905 100644 (file)
@@ -38,6 +38,7 @@ struct formatstring_parser *formatstring_parsers[NFORMATS] =
   /* format_objc */             &formatstring_objc,
   /* format_sh */               &formatstring_sh,
   /* format_python */           &formatstring_python,
+  /* format_python_brace */     &formatstring_python_brace,
   /* format_lisp */             &formatstring_lisp,
   /* format_elisp */            &formatstring_elisp,
   /* format_librep */           &formatstring_librep,
index 2297e6d500e56a261e279a75f98de9fa6fa5e069..d92532ddffe0ce1c169d3bd369365ed1e1e218c6 100644 (file)
@@ -99,6 +99,7 @@ extern DLL_VARIABLE struct formatstring_parser formatstring_c;
 extern DLL_VARIABLE struct formatstring_parser formatstring_objc;
 extern DLL_VARIABLE struct formatstring_parser formatstring_sh;
 extern DLL_VARIABLE struct formatstring_parser formatstring_python;
+extern DLL_VARIABLE struct formatstring_parser formatstring_python_brace;
 extern DLL_VARIABLE struct formatstring_parser formatstring_lisp;
 extern DLL_VARIABLE struct formatstring_parser formatstring_elisp;
 extern DLL_VARIABLE struct formatstring_parser formatstring_librep;
index 4e627761a4faa8a54305b6155afbe081b95dc8f8..586675f14a4676f4c296611f67a1cc55c8479e3e 100644 (file)
@@ -38,6 +38,7 @@ const char *const format_language[NFORMATS] =
   /* format_objc */             "objc",
   /* format_sh */               "sh",
   /* format_python */           "python",
+  /* format_python_brace */     "python-brace",
   /* format_lisp */             "lisp",
   /* format_elisp */            "elisp",
   /* format_librep */           "librep",
@@ -68,6 +69,7 @@ const char *const format_language_pretty[NFORMATS] =
   /* format_objc */             "Objective C",
   /* format_sh */               "Shell",
   /* format_python */           "Python",
+  /* format_python_brace */     "Python brace",
   /* format_lisp */             "Lisp",
   /* format_elisp */            "Emacs Lisp",
   /* format_librep */           "librep",
index 91e88a4834902f09b816f3615ae7c95ae5752fc0..bf2215ad73900f8609a352a85cc3b82ecfc496df 100644 (file)
@@ -47,6 +47,7 @@ enum format_type
   format_objc,
   format_sh,
   format_python,
+  format_python_brace,
   format_lisp,
   format_elisp,
   format_librep,
@@ -70,7 +71,7 @@ enum format_type
   format_lua,
   format_javascript
 };
-#define NFORMATS 26     /* Number of format_type enum values.  */
+#define NFORMATS 27     /* Number of format_type enum values.  */
 extern DLL_VARIABLE const char *const format_language[NFORMATS];
 extern DLL_VARIABLE const char *const format_language_pretty[NFORMATS];
 
index 14f8bc56e3bc5685eabe1fd871fa989feb5162a8..b70b048d7979707d19756b0a52aa894c51641282 100644 (file)
@@ -32,7 +32,7 @@ extern "C" {
 
 #define SCANNERS_PYTHON \
   { "Python",           extract_python,                                   \
-                        &flag_table_python, &formatstring_python, NULL }, \
+                        &flag_table_python, &formatstring_python, &formatstring_python_brace }, \
 
 /* Scan a Python file and add its translatable strings to mdlp.  */
 extern void extract_python (FILE *fp, const char *real_filename,
index 9ed419991ee031e704b8a2510b89235c7b6e1e69..d433794758a2f59d63327698c1124a99d6331a8c 100644 (file)
@@ -1688,6 +1688,11 @@ xgettext_record_flag (const char *optionstring)
                                                     name_start, name_end,
                                                     argnum, value, pass);
                     break;
+                  case format_python_brace:
+                    flag_context_list_table_insert (&flag_table_python, 0,
+                                                    name_start, name_end,
+                                                    argnum, value, pass);
+                    break;
                   case format_lisp:
                     flag_context_list_table_insert (&flag_table_lisp, 0,
                                                     name_start, name_end,
index c4e9a278209f6e9e744e9aaf5d40a2885cefb1c0..0a1c5d1bac4daaaffeb7f7b45601d3f13469ee5e 100644 (file)
@@ -1,3 +1,10 @@
+2013-04-26  Daiki Ueno  <ueno@gnu.org>
+
+       Support for Python brace format.
+       * format-python-brace-1: New file.
+       * format-python-brace-2: New file.
+       * Makefile.am (TESTS): Add them.
+
 2013-04-22  Daiki Ueno  <ueno@gnu.org>
 
        Support explicit string concatenation in Python.
index 8f7b52d071d5413501823ac1b8ce91492a0fd781..e0133b6464c32b5f2637a9d1c2544d55be1bb89b 100644 (file)
@@ -116,6 +116,7 @@ TESTS = gettext-1 gettext-2 gettext-3 gettext-4 gettext-5 gettext-6 gettext-7 \
        format-lisp-1 format-lisp-2 \
        format-php-1 format-php-2 \
        format-python-1 format-python-2 \
+       format-python-brace-1 format-python-brace-2 \
        format-pascal-1 format-pascal-2 \
        format-perl-1 format-perl-2 \
        format-perl-brace-1 format-perl-brace-2 \
diff --git a/gettext-tools/tests/format-python-brace-1 b/gettext-tools/tests/format-python-brace-1
new file mode 100755 (executable)
index 0000000..3446230
--- /dev/null
@@ -0,0 +1,73 @@
+#! /bin/sh
+
+# Test recognition of Python brace format strings.
+
+tmpfiles=""
+trap 'rm -fr $tmpfiles' 1 2 3 15
+
+tmpfiles="$tmpfiles f-pyb-1.data"
+cat <<\EOF > f-pyb-1.data
+# Invalid: no argument
+"abc"
+# Valid: a named argument
+"abc{value}"
+# Invalid: an empty name
+"abc{}"
+# Invalid: unterminated name
+"abc{value"
+# Valid: three arguments, two with equal names
+"abc{addr},{char},{addr}"
+# Valid: getattr operator
+"abc{value.name}"
+# Invalid: getattr operator with numeric field name
+"abc{value.0}"
+# Valid: getitem operator
+"abc{value[name]}"
+# Invalid: unterminated getitem operator
+"abc{value[name}"
+# Valid: format specifier
+"abc{value:0}"
+# Valid: nested format specifier
+"abc{value:0{foo}0}"
+# Invalid: too many nesting of format specifier
+"abc{value:0{foo:0}0}"
+EOF
+
+: ${XGETTEXT=xgettext}
+n=0
+while read comment; do
+  read string
+  n=`expr $n + 1`
+  tmpfiles="$tmpfiles f-pyb-1-$n.in f-pyb-1-$n.po"
+  cat <<EOF > f-pyb-1-$n.in
+gettext(${string});
+EOF
+  ${XGETTEXT} -L Python --flag=gettext:1:pass-python-brace-format -o f-pyb-1-$n.po f-pyb-1-$n.in || exit 1
+  test -f f-pyb-1-$n.po || exit 1
+  fail=
+  if echo "$comment" | grep 'Valid:' > /dev/null; then
+    if grep python-brace-format f-pyb-1-$n.po > /dev/null; then
+      :
+    else
+      fail=yes
+    fi
+  else
+    if grep python-brace-format f-pyb-1-$n.po > /dev/null; then
+      fail=yes
+    else
+      :
+    fi
+  fi
+  if test -n "$fail"; then
+    echo "Format string recognition error:" 1>&2
+    cat f-pyb-1-$n.in 1>&2
+    echo "Got:" 1>&2
+    cat f-pyb-1-$n.po 1>&2
+    exit 1
+  fi
+  rm -f f-pyb-1-$n.in f-pyb-1-$n.po
+done < f-pyb-1.data
+
+rm -fr $tmpfiles
+
+exit 0
diff --git a/gettext-tools/tests/format-python-brace-2 b/gettext-tools/tests/format-python-brace-2
new file mode 100755 (executable)
index 0000000..734baa3
--- /dev/null
@@ -0,0 +1,70 @@
+#! /bin/sh
+
+# Test checking of Python brace format strings.
+
+tmpfiles=""
+trap 'rm -fr $tmpfiles' 1 2 3 15
+
+tmpfiles="$tmpfiles f-pyb-2.data"
+cat <<\EOF > f-pyb-2.data
+# Valid: same named arguments
+msgid  "abc{date}{time}"
+msgstr "xyz{date}{time}"
+# Valid: permutation
+msgid  "abc{x3}{x1}{x2}def"
+msgstr "xyz{x2}{x1}{x3}"
+# Invalid: missing argument
+msgid  "abc{x2}def{x1}"
+msgstr "xyz{x1}"
+# Invalid: missing argument
+msgid  "abc{x1}def{x2}"
+msgstr "xyz{x2}"
+# Valid: added argument (valid since "{zoo}" expands to itself)
+msgid  "abc{foo}def"
+msgstr "xyz{foo}uvw{zoo}"
+# Valid: multiple reuse of same argument
+msgid  "{foo} {bar} {baz}"
+msgstr "{baz} {bar} {foo} {bar}"
+# Valid: single reuse of same argument
+msgid  "{baz} {bar} {foo} {bar}"
+msgstr "{foo} {bar} {baz}"
+EOF
+
+: ${MSGFMT=msgfmt}
+n=0
+while read comment; do
+  read msgid_line
+  read msgstr_line
+  n=`expr $n + 1`
+  tmpfiles="$tmpfiles f-pyb-2-$n.po f-pyb-2-$n.mo"
+  cat <<EOF > f-pyb-2-$n.po
+#, python-brace-format
+${msgid_line}
+${msgstr_line}
+EOF
+  fail=
+  if echo "$comment" | grep 'Valid:' > /dev/null; then
+    if ${MSGFMT} --check-format -o f-pyb-2-$n.mo f-pyb-2-$n.po; then
+      :
+    else
+      fail=yes
+    fi
+  else
+    ${MSGFMT} --check-format -o f-pyb-2-$n.mo f-pyb-2-$n.po 2> /dev/null
+    if test $? = 1; then
+      :
+    else
+      fail=yes
+    fi
+  fi
+  if test -n "$fail"; then
+    echo "Format string checking error:" 1>&2
+    cat f-pyb-2-$n.po 1>&2
+    exit 1
+  fi
+  rm -f f-pyb-2-$n.po f-pyb-2-$n.mo
+done < f-pyb-2.data
+
+rm -fr $tmpfiles
+
+exit 0
index e37b178315300f7d5c419f15071db0829abee268..d4bf02133efa7cd2ed33e31dd0e648936985695f 100644 (file)
@@ -42,6 +42,7 @@ VARIABLE(formatstring_perl)
 VARIABLE(formatstring_perl_brace)
 VARIABLE(formatstring_php)
 VARIABLE(formatstring_python)
+VARIABLE(formatstring_python_brace)
 VARIABLE(formatstring_qt)
 VARIABLE(formatstring_qt_plural)
 VARIABLE(formatstring_scheme)