This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix typos in LC_TELEPHONE in i18n locale


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.

-- 
Mike FABIAN <mfabian@redhat.com>


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]