Attachment 10426 Details for Bug 10871 – Implement alternative month names (v9)

[patch] Implement alternative month names (v9)

0002-Implement-alternative-month-names-bug-10871.patch (text/plain), 14.08 KB, created by Rafal Luzynski on 2017-09-19 10:10:26 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Rafal Luzynski

Created: 2017-09-19 10:10:26 UTC

Size: 14.08 KB

patch

obsolete

>From df6a28ed309c26588533ccfa04019b03135650d0 Mon Sep 17 00:00:00 2001
>From: Rafal Luzynski <digitalfreak@lingonborough.com>
>Date: Thu, 19 Nov 2015 01:40:14 +0100
>Subject: [PATCH 2/6] Implement alternative month names (bug 10871).
>
>Some languages (Slavic, Baltic, etc.) require a genitive case of the
>month name when formatting a full date (with the day number) while
>they require a nominative case when referring to the month standalone.
>This requirement cannot be fulfilled without providing two forms for
>each month name.  From now it is precised that nl_langinfo(MON_1)
>series (up to MON_12) and strftime("%B") generate the month names in
>the grammatical form used when the month forms part of a complete date.
>If the grammatical form used when the month is named by itself is needed,
>the new values nl_langinfo(ALTMON_1) (up to ALTMON_12) and
>strftime("%OB") are supported.  This new feature is optional so the
>languages which do not need it or do not yet provide the updated
>locales simply do not use it and their behaviour is unchanged.
>
>	[BZ #10871]
>	* locale/C-time.c: Add alternative month names, define them as the
>	same as mon explicitly.
>	* locale/categories.def: alt_mon and wide-alt_mon added.
>	* locale/langinfo.h: ALTMON_1 .. ALTMON_12 and similar contants
>	defined.
>	* locale/programs/ld-time.c: Alternative month names support
>	added, they are a copy of mon if not specified explicitly.
>	* locale/programs/locfile-kw.gperf: alt_mon defined.
>	* locale/programs/locfile-token.h: tok_alt_mon defined.
>	* localedata/tst-langinfo.c: Add tests for the new constants
>	ALTMON_1 .. ALTMON_12.
>	* time/strftime_l.c: %OB format for alternative month names
>	added.
>	* time/strptime_l.c: Alternative month names also recognized.
>---
> ChangeLog                        | 18 +++++++++++++++
> locale/C-time.c                  | 28 ++++++++++++++++++++--
> locale/categories.def            |  2 ++
> locale/langinfo.h                | 50 ++++++++++++++++++++++++++++++++++++++--
> locale/programs/ld-time.c        | 21 +++++++++++++++++
> locale/programs/locfile-kw.gperf |  1 +
> locale/programs/locfile-token.h  |  1 +
> localedata/tst-langinfo.c        | 12 ++++++++++
> time/strftime_l.c                | 11 +++++++--
> time/strptime_l.c                | 24 +++++++++++++++++++
> 10 files changed, 162 insertions(+), 6 deletions(-)
>
>diff --git a/ChangeLog b/ChangeLog
>index 3b8e6c5..b0636ff 100644
>--- a/ChangeLog
>+++ b/ChangeLog
>@@ -1,5 +1,23 @@
> 2017-09-19  Rafal Luzynski  <digitalfreak@lingonborough.com>
> 
>+	[BZ #10871]
>+	* locale/C-time.c: Add alternative month names, define them as the
>+	same as mon explicitly.
>+	* locale/categories.def: alt_mon and wide-alt_mon added.
>+	* locale/langinfo.h: ALTMON_1 .. ALTMON_12 and similar contants
>+	defined.
>+	* locale/programs/ld-time.c: Alternative month names support
>+	added, they are a copy of mon if not specified explicitly.
>+	* locale/programs/locfile-kw.gperf: alt_mon defined.
>+	* locale/programs/locfile-token.h: tok_alt_mon defined.
>+	* localedata/tst-langinfo.c: Add tests for the new constants
>+	ALTMON_1 .. ALTMON_12.
>+	* time/strftime_l.c: %OB format for alternative month names
>+	added.
>+	* time/strptime_l.c: Alternative month names also recognized.
>+
>+2017-09-19  Rafal Luzynski  <digitalfreak@lingonborough.com>
>+
> 	* locale/loadlocale.c: Correct size of
> 	_nl_value_type_LC_<category> arrays.
> 
>diff --git a/locale/C-time.c b/locale/C-time.c
>index 31d8704..ee33652 100644
>--- a/locale/C-time.c
>+++ b/locale/C-time.c
>@@ -30,7 +30,7 @@ const struct __locale_data _nl_C_LC_TIME attribute_hidden =
>   { NULL, },			/* no cached data */
>   UNDELETABLE,
>   0,
>-  111,
>+  135,
>   {
>     { .string = "Sun" },
>     { .string = "Mon" },
>@@ -142,6 +142,30 @@ const struct __locale_data _nl_C_LC_TIME attribute_hidden =
>     { .string = "" },
>     { .string = "%a %b %e %H:%M:%S %Z %Y" },
>     { .wstr = (const uint32_t *) L"%a %b %e %H:%M:%S %Z %Y" },
>-    { .string = _nl_C_codeset }
>+    { .string = _nl_C_codeset },
>+    { .string = "January" },
>+    { .string = "February" },
>+    { .string = "March" },
>+    { .string = "April" },
>+    { .string = "May" },
>+    { .string = "June" },
>+    { .string = "July" },
>+    { .string = "August" },
>+    { .string = "September" },
>+    { .string = "October" },
>+    { .string = "November" },
>+    { .string = "December" },
>+    { .wstr = (const uint32_t *) L"January" },
>+    { .wstr = (const uint32_t *) L"February" },
>+    { .wstr = (const uint32_t *) L"March" },
>+    { .wstr = (const uint32_t *) L"April" },
>+    { .wstr = (const uint32_t *) L"May" },
>+    { .wstr = (const uint32_t *) L"June" },
>+    { .wstr = (const uint32_t *) L"July" },
>+    { .wstr = (const uint32_t *) L"August" },
>+    { .wstr = (const uint32_t *) L"September" },
>+    { .wstr = (const uint32_t *) L"October" },
>+    { .wstr = (const uint32_t *) L"November" },
>+    { .wstr = (const uint32_t *) L"December" }
>   }
> };
>diff --git a/locale/categories.def b/locale/categories.def
>index 27a6129..53ec8c5 100644
>--- a/locale/categories.def
>+++ b/locale/categories.def
>@@ -249,6 +249,8 @@ DEFINE_CATEGORY
>   DEFINE_ELEMENT (_DATE_FMT,                "date_fmt",            opt, string)
>   DEFINE_ELEMENT (_NL_W_DATE_FMT,           "wide-date_fmt",       opt, wstring)
>   DEFINE_ELEMENT (_NL_TIME_CODESET,	    "time-codeset",	   std, string)
>+  DEFINE_ELEMENT (ALTMON_1,       "alt_mon",       opt, stringarray, 12, 12)
>+  DEFINE_ELEMENT (_NL_WALTMON_1,  "wide-alt_mon",  opt, wstringarray, 12, 12)
>   ), NO_POSTLOAD)
> 
> 
>diff --git a/locale/langinfo.h b/locale/langinfo.h
>index 1403957..78103ce 100644
>--- a/locale/langinfo.h
>+++ b/locale/langinfo.h
>@@ -100,7 +100,8 @@ enum
>   ABMON_12,
> #define ABMON_12		ABMON_12
> 
>-  /* Long month names.  */
>+  /* Long month names, in the grammatical form used when the month
>+     forms part of a complete date.  */
>   MON_1,			/* January */
> #define MON_1			MON_1
>   MON_2,
>@@ -189,7 +190,8 @@ enum
>   _NL_WABMON_11,
>   _NL_WABMON_12,
> 
>-  /* Long month names.  */
>+  /* Long month names, in the grammatical form used when the month
>+     forms part of a complete date.  */
>   _NL_WMON_1,		/* January */
>   _NL_WMON_2,
>   _NL_WMON_3,
>@@ -231,6 +233,50 @@ enum
> 
>   _NL_TIME_CODESET,
> 
>+  /* Long month names, in the grammatical form used when the month
>+     is named by itself.  */
>+  __ALTMON_1,			/* January */
>+  __ALTMON_2,
>+  __ALTMON_3,
>+  __ALTMON_4,
>+  __ALTMON_5,
>+  __ALTMON_6,
>+  __ALTMON_7,
>+  __ALTMON_8,
>+  __ALTMON_9,
>+  __ALTMON_10,
>+  __ALTMON_11,
>+  __ALTMON_12,
>+#ifdef __USE_GNU
>+# define ALTMON_1		__ALTMON_1
>+# define ALTMON_2		__ALTMON_2
>+# define ALTMON_3		__ALTMON_3
>+# define ALTMON_4		__ALTMON_4
>+# define ALTMON_5		__ALTMON_5
>+# define ALTMON_6		__ALTMON_6
>+# define ALTMON_7		__ALTMON_7
>+# define ALTMON_8		__ALTMON_8
>+# define ALTMON_9		__ALTMON_9
>+# define ALTMON_10		__ALTMON_10
>+# define ALTMON_11		__ALTMON_11
>+# define ALTMON_12		__ALTMON_12
>+#endif
>+
>+  /* Long month names, in the grammatical form used when the month
>+     is named by itself.  */
>+  _NL_WALTMON_1,			/* January */
>+  _NL_WALTMON_2,
>+  _NL_WALTMON_3,
>+  _NL_WALTMON_4,
>+  _NL_WALTMON_5,
>+  _NL_WALTMON_6,
>+  _NL_WALTMON_7,
>+  _NL_WALTMON_8,
>+  _NL_WALTMON_9,
>+  _NL_WALTMON_10,
>+  _NL_WALTMON_11,
>+  _NL_WALTMON_12,
>+
>   _NL_NUM_LC_TIME,	/* Number of indices in LC_TIME category.  */
> 
>   /* LC_COLLATE category: text sorting.
>diff --git a/locale/programs/ld-time.c b/locale/programs/ld-time.c
>index 32e9c41..0383179 100644
>--- a/locale/programs/ld-time.c
>+++ b/locale/programs/ld-time.c
>@@ -91,6 +91,9 @@ struct locale_time_t
>   const char *date_fmt;
>   const uint32_t *wdate_fmt;
>   int alt_digits_defined;
>+  const char *alt_mon[12];
>+  const uint32_t *walt_mon[12];
>+  int alt_mon_defined;
>   unsigned char week_ndays;
>   uint32_t week_1stday;
>   unsigned char week_1stweek;
>@@ -652,6 +655,15 @@ time_output (struct localedef_t *locale, const struct charmap_t *charmap,
>   add_locale_string (&file, time->date_fmt);
>   add_locale_wstring (&file, time->wdate_fmt);
>   add_locale_string (&file, charmap->code_set_name);
>+
>+  /* The alt'mons.  */
>+  for (n = 0; n < 12; ++n)
>+    add_locale_string (&file, time->alt_mon[n] ?: "");
>+
>+  /* The wide character alt'mons.  */
>+  for (n = 0; n < 12; ++n)
>+    add_locale_wstring (&file, time->walt_mon[n] ?: empty_wstr);
>+
>   write_locale_data (output_path, LC_TIME, "LC_TIME", &file);
> }
> 
>@@ -795,6 +807,7 @@ time_read (struct linereader *ldfile, struct localedef_t *result,
> 	  STRARR_ELEM (mon, 12, 12);
> 	  STRARR_ELEM (am_pm, 2, 2);
> 	  STRARR_ELEM (alt_digits, 0, 100);
>+	  STRARR_ELEM (alt_mon, 12, 12);
> 
> 	case tok_era:
> 	  /* Ignore the rest of the line if we don't need the input of
>@@ -947,6 +960,14 @@ time_read (struct linereader *ldfile, struct localedef_t *result,
> 	    lr_error (ldfile, _("\
> %1$s: definition does not end with `END %1$s'"), "LC_TIME");
> 	  lr_ignore_rest (ldfile, now->tok == tok_lc_time);
>+
>+	  /* If alt_mon was not specified, make it a copy of mon.  */
>+	  if (!ignore_content && !time->alt_mon_defined)
>+	    {
>+	      memcpy (time->alt_mon, time->mon, sizeof (time->mon));
>+	      memcpy (time->walt_mon, time->wmon, sizeof (time->wmon));
>+	      time->alt_mon_defined = 1;
>+	    }
> 	  return;
> 
> 	default:
>diff --git a/locale/programs/locfile-kw.gperf b/locale/programs/locfile-kw.gperf
>index 3605d15..3d11cc6 100644
>--- a/locale/programs/locfile-kw.gperf
>+++ b/locale/programs/locfile-kw.gperf
>@@ -148,6 +148,7 @@ first_workday,          tok_first_workday,          0
> cal_direction,          tok_cal_direction,          0
> timezone,               tok_timezone,               0
> date_fmt,               tok_date_fmt,               0
>+alt_mon,                tok_alt_mon,                0
> LC_MESSAGES,            tok_lc_messages,            0
> yesexpr,                tok_yesexpr,                0
> noexpr,                 tok_noexpr,                 0
>diff --git a/locale/programs/locfile-token.h b/locale/programs/locfile-token.h
>index 0c32f2c..2a313b2 100644
>--- a/locale/programs/locfile-token.h
>+++ b/locale/programs/locfile-token.h
>@@ -186,6 +186,7 @@ enum token_t
>   tok_cal_direction,
>   tok_timezone,
>   tok_date_fmt,
>+  tok_alt_mon,
>   tok_lc_messages,
>   tok_yesexpr,
>   tok_noexpr,
>diff --git a/localedata/tst-langinfo.c b/localedata/tst-langinfo.c
>index 1012f56..c23d9e0 100644
>--- a/localedata/tst-langinfo.c
>+++ b/localedata/tst-langinfo.c
>@@ -50,6 +50,18 @@ struct map
>   VAL (ABMON_8),
>   VAL (ABMON_9),
>   VAL (ALT_DIGITS),
>+  VAL (ALTMON_1),
>+  VAL (ALTMON_10),
>+  VAL (ALTMON_11),
>+  VAL (ALTMON_12),
>+  VAL (ALTMON_2),
>+  VAL (ALTMON_3),
>+  VAL (ALTMON_4),
>+  VAL (ALTMON_5),
>+  VAL (ALTMON_6),
>+  VAL (ALTMON_7),
>+  VAL (ALTMON_8),
>+  VAL (ALTMON_9),
>   VAL (AM_STR),
>   VAL (CRNCYSTR),
>   VAL (CURRENCY_SYMBOL),
>diff --git a/time/strftime_l.c b/time/strftime_l.c
>index b5ba9ca..1c4bed8 100644
>--- a/time/strftime_l.c
>+++ b/time/strftime_l.c
>@@ -492,6 +492,9 @@ __strftime_internal (CHAR_T *s, size_t maxsize, const CHAR_T *format,
> # define f_month \
>   ((const CHAR_T *) (tp->tm_mon < 0 || tp->tm_mon > 11			     \
> 		     ? "?" : _NL_CURRENT (LC_TIME, NLW(MON_1) + tp->tm_mon)))
>+# define f_altmonth \
>+  ((const CHAR_T *) (tp->tm_mon < 0 || tp->tm_mon > 11			     \
>+		     ? "?" : _NL_CURRENT (LC_TIME, NLW(ALTMON_1) + tp->tm_mon)))
> # define ampm \
>   ((const CHAR_T *) _NL_CURRENT (LC_TIME, tp->tm_hour > 11		      \
> 				 ? NLW(PM_STR) : NLW(AM_STR)))
>@@ -507,6 +510,7 @@ __strftime_internal (CHAR_T *s, size_t maxsize, const CHAR_T *format,
> 		   ? "?" : month_name[tp->tm_mon])
> #  define a_wkday f_wkday
> #  define a_month f_month
>+#  define f_altmonth f_month
> #  define ampm (L_("AMPM") + 2 * (tp->tm_hour > 11))
> 
>   size_t aw_len = 3;
>@@ -785,7 +789,7 @@ __strftime_internal (CHAR_T *s, size_t maxsize, const CHAR_T *format,
> #endif
> 
> 	case L_('B'):
>-	  if (modifier != 0)
>+	  if (modifier == L_('E'))
> 	    goto bad_format;
> 	  if (change_case)
> 	    {
>@@ -793,7 +797,10 @@ __strftime_internal (CHAR_T *s, size_t maxsize, const CHAR_T *format,
> 	      to_lowcase = 0;
> 	    }
> #if defined _NL_CURRENT || !HAVE_STRFTIME
>-	  cpy (STRLEN (f_month), f_month);
>+	  if (modifier == L_('O'))
>+	    cpy (STRLEN (f_altmonth), f_altmonth);
>+	  else
>+	    cpy (STRLEN (f_month), f_month);
> 	  break;
> #else
> 	  goto underlying_strftime;
>diff --git a/time/strptime_l.c b/time/strptime_l.c
>index 3afc33a..b99f5d2 100644
>--- a/time/strptime_l.c
>+++ b/time/strptime_l.c
>@@ -124,6 +124,8 @@ extern const struct __locale_data _nl_C_LC_TIME attribute_hidden;
>   (&_nl_C_LC_TIME.values[_NL_ITEM_INDEX (ABDAY_1)].string)
> # define month_name (&_nl_C_LC_TIME.values[_NL_ITEM_INDEX (MON_1)].string)
> # define ab_month_name (&_nl_C_LC_TIME.values[_NL_ITEM_INDEX (ABMON_1)].string)
>+# define alt_month_name \
>+  (&_nl_C_LC_TIME.values[_NL_ITEM_INDEX (ALTMON_1)].string)
> # define HERE_D_T_FMT (_nl_C_LC_TIME.values[_NL_ITEM_INDEX (D_T_FMT)].string)
> # define HERE_D_FMT (_nl_C_LC_TIME.values[_NL_ITEM_INDEX (D_FMT)].string)
> # define HERE_AM_STR (_nl_C_LC_TIME.values[_NL_ITEM_INDEX (AM_STR)].string)
>@@ -402,6 +404,20 @@ __strptime_internal (const char *rp, const char *fmt, struct tm *tmp,
> 	      if (s.decided !=raw)
> 		{
> 		  trp = rp;
>+#ifdef _LIBC
>+		  /* First check the alt month.  */
>+		  if (match_string (_NL_CURRENT (LC_TIME, ALTMON_1 + cnt), trp)
>+		      && trp > rp_longest)
>+		    {
>+		      rp_longest = trp;
>+		      cnt_longest = cnt;
>+		      if (s.decided == not
>+			  && strcmp (_NL_CURRENT (LC_TIME, ALTMON_1 + cnt),
>+				     alt_month_name[cnt]))
>+			decided_longest = loc;
>+		    }
>+		  trp = rp;
>+#endif
> 		  if (match_string (_NL_CURRENT (LC_TIME, MON_1 + cnt), trp)
> 		      && trp > rp_longest)
> 		    {
>@@ -428,6 +444,10 @@ __strptime_internal (const char *rp, const char *fmt, struct tm *tmp,
> 	      if (s.decided != loc
> 		  && (((trp = rp, match_string (month_name[cnt], trp))
> 		       && trp > rp_longest)
>+#ifdef _LIBC
>+		      || ((trp = rp, match_string (alt_month_name[cnt], trp))
>+			  && trp > rp_longest)
>+#endif
> 		      || ((trp = rp, match_string (ab_month_name[cnt], trp))
> 			  && trp > rp_longest)))
> 		{
>@@ -1015,6 +1035,10 @@ __strptime_internal (const char *rp, const char *fmt, struct tm *tmp,
> 	case 'O':
> 	  switch (*fmt++)
> 	    {
>+	    case 'B':
>+	      /* Undo the increment and continue.  */
>+	      fmt--;
>+	      break;
> 	    case 'd':
> 	    case 'e':
> 	      /* Match day of month using alternate numeric symbols.  */
>-- 
>2.7.5
>

Actions: View | Diff

Attachments on bug 10871: 8795 | 8874 | 8875 | 8876 | 8877 | 8878 | 8879 | 8880 | 9122 | 9123 | 9124 | 9125 | 9126 | 9127 | 9569 | 9570 | 9571 | 9572 | 9573 | 9574 | 9575 | 9576 | 9577 | 9578 | 9579 | 9595 | 9596 | 9597 | 9598 | 9599 | 9709 | 9710 | 9711 | 9712 | 9713 | 9714 | 9715 | 9716 | 9717 | 9718 | 9719 | 9720 | 9721 | 9908 | 9909 | 9910 | 9911 | 9912 | 9913 | 9914 | 9915 | 9916 | 9917 | 9918 | 9919 | 10063 | 10064 | 10065 | 10225 | 10226 | 10227 | 10228 | 10229 | 10230 | 10231 | 10425 | 10426 | 10427 | 10428 | 10429 | 10430 | 10592 | 10593 | 10594 | 10595 | 10596 | 10597 | 10598 | 10599 | 10600 | 10601 | 10602 | 10731 | 10732 | 10733 | 10734 | 10735 | 10736 | 10737 | 10738 | 10739 | 10740 | 10741 | 10742