Fix typos in LC_TELEPHONE in i18n locale

  There seem to be two typos in tel_int_fmt in LC_TELEPHONE
in glibc/localedata/locales/i18n. The attached patch fixes this.
  

On 01 Nov 2015 09:53, Mike FABIAN wrote:
>     * locales/i18n: fix typos (tel_int_fmt "+%c +a +l" -> "+%c %a %l")

this looks correct

>  % This is the ISO/IEC 14652 "i18n" definition for
>  % the LC_TELEPHONE category.
>  %
> -tel_int_fmt    "<U002B><U0025><U0063><U0020><U002B><U0061><U0020><U002B>/
> +tel_int_fmt    "<U002B><U0025><U0063><U0020><U0025><U0061><U0020><U0025>/
>  <U006C>"

when i look up the spec, i find:
http://www.open-std.org/jtc1/SC22/WG20/docs/n972-14652ft.pdf

% "+%c %a%t%l"
tel_int_fmt "<U002B><U0025><U0063><U0020><U0025><U0061><U0025><U0074><U0025><U006C>"

any idea why it says to use %t but we use a space instead ?
-mike
  

Mike Frysinger <vapier@gentoo.org> wrote:

> On 01 Nov 2015 09:53, Mike FABIAN wrote:
>>     * locales/i18n: fix typos (tel_int_fmt "+%c +a +l" -> "+%c %a %l")
>
> this looks correct
>
>>  % This is the ISO/IEC 14652 "i18n" definition for
>>  % the LC_TELEPHONE category.
>>  %
>> -tel_int_fmt    "<U002B><U0025><U0063><U0020><U002B><U0061><U0020><U002B>/
>> +tel_int_fmt    "<U002B><U0025><U0063><U0020><U0025><U0061><U0020><U0025>/
>>  <U006C>"
>
> when i look up the spec, i find:
> http://www.open-std.org/jtc1/SC22/WG20/docs/n972-14652ft.pdf
>
> % "+%c %a%t%l"
> tel_int_fmt
> "<U002B><U0025><U0063><U0020><U0025><U0061><U0025><U0074><U0025><U006C>"
>
> any idea why it says to use %t but we use a space instead ?
> -mike

I have no idea either. The spec you quote says:

%a area code without nationwide prefix (prefix is often <0>).
%A area code including nationwide prefix (prefix is often <0>).
%l local number (within area code).
%e extension (to local number)
%c country code
%C alternate carrier service code used for dialling abroad
%t Insert a <space> if the previous descriptor’s value was not an empty
   string; otherwise ignore.

So the %t would make a difference if %a happens to be empty, it
would avoid a double space then. I don't know whether it is possible
for %a to be empty. Sounds weird, are there countries which have
no area code?

Shall I make a new patch with the %t? The %t cannot hurt.
  

On Tue, Nov 03, 2015 at 02:19:45PM +0900, Mike FABIAN wrote:
> Mike Frysinger <vapier@gentoo.org> wrote:
> 
> > On 01 Nov 2015 09:53, Mike FABIAN wrote:
> >>     * locales/i18n: fix typos (tel_int_fmt "+%c +a +l" -> "+%c %a %l")
> >
> > this looks correct
> >
> >>  % This is the ISO/IEC 14652 "i18n" definition for
> >>  % the LC_TELEPHONE category.
> >>  %
> >> -tel_int_fmt    "<U002B><U0025><U0063><U0020><U002B><U0061><U0020><U002B>/
> >> +tel_int_fmt    "<U002B><U0025><U0063><U0020><U0025><U0061><U0020><U0025>/
> >>  <U006C>"
> >
> > when i look up the spec, i find:
> > http://www.open-std.org/jtc1/SC22/WG20/docs/n972-14652ft.pdf
> >
> > % "+%c %a%t%l"
> > tel_int_fmt
> > "<U002B><U0025><U0063><U0020><U0025><U0061><U0025><U0074><U0025><U006C>"
> >
> > any idea why it says to use %t but we use a space instead ?
> > -mike
> 
> I have no idea either. The spec you quote says:
> 
> %a area code without nationwide prefix (prefix is often <0>).
> %A area code including nationwide prefix (prefix is often <0>).
> %l local number (within area code).
> %e extension (to local number)
> %c country code
> %C alternate carrier service code used for dialling abroad
> %t Insert a <space> if the previous descriptor???s value was not an empty
>    string; otherwise ignore.
> 
> So the %t would make a difference if %a happens to be empty, it
> would avoid a double space then. I don't know whether it is possible
> for %a to be empty. Sounds weird, are there countries which have
> no area code?
> 
> Shall I make a new patch with the %t? The %t cannot hurt.

Yes, there are countries without area codes, such as Denmark and Norway.

Best regards
keld
  

keld@keldix.com wrote:

> On Tue, Nov 03, 2015 at 02:19:45PM +0900, Mike FABIAN wrote:
>> Mike Frysinger <vapier@gentoo.org> wrote:
>> 
>> > On 01 Nov 2015 09:53, Mike FABIAN wrote:
>> >>     * locales/i18n: fix typos (tel_int_fmt "+%c +a +l" -> "+%c %a %l")
>> >
>> > this looks correct
>> >
>> >>  % This is the ISO/IEC 14652 "i18n" definition for
>> >>  % the LC_TELEPHONE category.
>> >>  %
>> >> -tel_int_fmt    "<U002B><U0025><U0063><U0020><U002B><U0061><U0020><U002B>/
>> >> +tel_int_fmt    "<U002B><U0025><U0063><U0020><U0025><U0061><U0020><U0025>/
>> >>  <U006C>"
>> >
>> > when i look up the spec, i find:
>> > http://www.open-std.org/jtc1/SC22/WG20/docs/n972-14652ft.pdf
>> >
>> > % "+%c %a%t%l"
>> > tel_int_fmt
>> > "<U002B><U0025><U0063><U0020><U0025><U0061><U0025><U0074><U0025><U006C>"
>> >
>> > any idea why it says to use %t but we use a space instead ?
>> > -mike
>> 
>> I have no idea either. The spec you quote says:
>> 
>> %a area code without nationwide prefix (prefix is often <0>).
>> %A area code including nationwide prefix (prefix is often <0>).
>> %l local number (within area code).
>> %e extension (to local number)
>> %c country code
>> %C alternate carrier service code used for dialling abroad
>> %t Insert a <space> if the previous descriptor???s value was not an empty
>>    string; otherwise ignore.
>> 
>> So the %t would make a difference if %a happens to be empty, it
>> would avoid a double space then. I don't know whether it is possible
>> for %a to be empty. Sounds weird, are there countries which have
>> no area code?
>> 
>> Shall I make a new patch with the %t? The %t cannot hurt.
>
> Yes, there are countries without area codes, such as Denmark and Norway.

Good, I’ll make a new patch adding the %t then.
  

On 03 Nov 2015 16:44, Mike FABIAN wrote:
> Good, I’ll make a new patch adding the %t then.

were you able to get around to this ? :)
-mike
  

Mike Frysinger <vapier@gentoo.org> さんはかきました:

> On 03 Nov 2015 16:44, Mike FABIAN wrote:
>> Good, I’ll make a new patch adding the %t then.
>
> were you able to get around to this ? :)
> -mike

I thought I did, but apparently I forgot to I posted the patch now.