Message ID | 1501243954-46361-1-git-send-email-akhilesh.k@samsung.com |
---|---|
State | Rejected |
Delegated to: | Mike Fabian |
Headers |
Received: (qmail 7654 invoked by alias); 28 Jul 2017 12:17:00 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <libc-alpha.sourceware.org> List-Unsubscribe: <mailto:libc-alpha-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:libc-alpha-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/libc-alpha/> List-Post: <mailto:libc-alpha@sourceware.org> List-Help: <mailto:libc-alpha-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 126601 invoked by uid 89); 28 Jul 2017 12:16:51 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RP_MATCHES_RCVD, SPF_HELO_PASS, SPF_PASS, UPPERCASE_50_75 autolearn=ham version=3.3.2 spammy=CHINA, REPUBLIC, arab, syrian X-HELO: mailout3.samsung.com From: Akhilesh Kumar <akhilesh.k@samsung.com> To: libc-alpha@sourceware.org Cc: Akhilesh Kumar <akhilesh.k@samsung.com> Subject: [PATCH 1/1] Fix iso 3166.def for TIWAN Date: Fri, 28 Jul 2017 17:42:34 +0530 Message-Id: <1501243954-46361-1-git-send-email-akhilesh.k@samsung.com> X-CMS-MailID: 20170728121634epcas5p3db3df5730c42459ebe67d112f0a38c3f X-Msg-Generator: CA X-Sender-IP: 182.195.40.13 X-Local-Sender: =?UTF-8?B?QUtISUxFU0ggS1VNQVIbU1JJLURlbGhpLVBsYXRmb3JtIFMv?= =?UTF-8?B?VyAxIFRlYW0b7IK87ISx7KCE7J6QG0NoaWVmIEVuZ2luZWVy?= X-Global-Sender: =?UTF-8?B?QUtISUxFU0ggS1VNQVIbU1JJLURlbGhpLVBsYXRmb3JtIFMv?= =?UTF-8?B?VyAxIFRlYW0bU2Ftc3VuZ8KgRWxlY3Ryb25pY3MbQ2hpZWYgRW5naW5lZXI=?= X-Sender-Code: =?UTF-8?B?QzEwG1NXQUhRG0MxMElEMDJJRDAyODExNQ==?= Content-Type: text/plain; charset="utf-8" X-MTR: 20170728121634epcas5p3db3df5730c42459ebe67d112f0a38c3f CMS-TYPE: 105P DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20170728121634epcas5p3db3df5730c42459ebe67d112f0a38c3f X-RootMTR: 20170728121634epcas5p3db3df5730c42459ebe67d112f0a38c3f References: <CGME20170728121634epcas5p3db3df5730c42459ebe67d112f0a38c3f@epcas5p3.samsung.com> |
Commit Message
Akhilesh Kumar
July 28, 2017, 12:12 p.m. UTC
Since TIWAN is the part for Republic Of China Made symmetry between SINT MAARTEN,SAINT MARTIN and TIWAN https://www.iso.org/obp/ui/#search "Page 9" [#BZ 21858] iso-3166.def : Fix English short Name for TIWAN --- locale/iso-3166.def | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
Comments
Akhilesh Kumar <akhilesh.k@samsung.com> wrote: > Since TIWAN is the part for Republic Of China > Made symmetry between SINT MAARTEN,SAINT MARTIN and TIWAN > https://www.iso.org/obp/ui/#search "Page 9" > > [#BZ 21858] > iso-3166.def : Fix English short Name for TIWAN > --- > locale/iso-3166.def | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/locale/iso-3166.def b/locale/iso-3166.def > index 52997b4..653a038 100644 > --- a/locale/iso-3166.def > +++ b/locale/iso-3166.def > @@ -224,7 +224,7 @@ DEFINE_COUNTRY_CODE ("SWAZILAND", SZ, SWZ, 748) > DEFINE_COUNTRY_CODE ("SWEDEN", SE, SWE, 752) > DEFINE_COUNTRY_CODE ("SWITZERLAND", CH, CHE, 756) > DEFINE_COUNTRY_CODE ("SYRIAN ARAB REPUBLIC", SY, SYR, 760) > -DEFINE_COUNTRY_CODE ("TAIWAN, PROVINCE OF CHINA", TW, TWN, 158) > +DEFINE_COUNTRY_CODE ("TAIWAN (PROVINCE OF CHINA)", TW, TWN, 158) I don’t really see why that is an improvement. > DEFINE_COUNTRY_CODE ("TAJIKISTAN", TJ, TJK, 762) > DEFINE_COUNTRY_CODE ("TANZANIA, UNITED REPUBLIC OF", TZ, TZA, 834) > DEFINE_COUNTRY_CODE ("THAILAND", TH, THA, 764)
> > @@ -224,7 +224,7 @@ DEFINE_COUNTRY_CODE ("SWAZILAND", SZ, SWZ, 748) > > DEFINE_COUNTRY_CODE ("SWEDEN", SE, SWE, 752) > > DEFINE_COUNTRY_CODE ("SWITZERLAND", CH, CHE, 756) > > DEFINE_COUNTRY_CODE ("SYRIAN ARAB REPUBLIC", SY, SYR, 760) > > -DEFINE_COUNTRY_CODE ("TAIWAN, PROVINCE OF CHINA", TW, TWN, 158) > > +DEFINE_COUNTRY_CODE ("TAIWAN (PROVINCE OF CHINA)", TW, TWN, 158) > > I don’t really see why that is an improvement. This patch only make consistency Like DEFINE_COUNTRY_CODE ("SINT MAARTEN (DUTCH PART)", SX, SXM, 534) DEFINE_COUNTRY_CODE ("SAINT MARTIN (FRENCH PART)", MF, MAF, 663) Query:- 1- > looks some countries info are missing Like “KOSOVA” ? 2- > Also Fron 1997 Hong Kong a sovereign state for “China” why country_num is different for “Hong Kong”, "China" and TAIWAN ? Some is found for other countries 3- >I believe this data is taken from iso.org is it mandatory ? > > DEFINE_COUNTRY_CODE ("TAJIKISTAN", TJ, TJK, 762) > > DEFINE_COUNTRY_CODE ("TANZANIA, UNITED REPUBLIC OF", TZ, TZA, 834) > > DEFINE_COUNTRY_CODE ("THAILAND", TH, THA, 764)
AKHILESH KUMAR <akhilesh.k@samsung.com> wrote: >> > @@ -224,7 +224,7 @@ DEFINE_COUNTRY_CODE ("SWAZILAND", SZ, SWZ, 748) >> > DEFINE_COUNTRY_CODE ("SWEDEN", SE, SWE, 752) >> > DEFINE_COUNTRY_CODE ("SWITZERLAND", CH, CHE, 756) >> > DEFINE_COUNTRY_CODE ("SYRIAN ARAB REPUBLIC", SY, SYR, 760) >> > -DEFINE_COUNTRY_CODE ("TAIWAN, PROVINCE OF CHINA", TW, TWN, 158) >> > +DEFINE_COUNTRY_CODE ("TAIWAN (PROVINCE OF CHINA)", TW, TWN, 158) >> >> I don’t really see why that is an improvement. > > > This patch only make consistency > Like > DEFINE_COUNTRY_CODE ("SINT MAARTEN (DUTCH PART)", SX, SXM, 534) > DEFINE_COUNTRY_CODE ("SAINT MARTIN (FRENCH PART)", MF, MAF, 663) > > > Query:- > 1- > looks some countries info are missing Like “KOSOVA” ? Maybe we don’t need an entry for Kosovo if we don’t have a locale for Kosovo anyway at the moment.
>>> > DEFINE_COUNTRY_CODE ("SWITZERLAND", CH, CHE, 756) >>> > DEFINE_COUNTRY_CODE ("SYRIAN ARAB REPUBLIC", SY, SYR, 760) >>> > -DEFINE_COUNTRY_CODE ("TAIWAN, PROVINCE OF CHINA", TW, TWN, 158) >>> > +DEFINE_COUNTRY_CODE ("TAIWAN (PROVINCE OF CHINA)", TW, TWN, 158) >>> >>> I don’t really see why that is an improvement. >> >> >> This patch only make consistency >> Like >> DEFINE_COUNTRY_CODE ("SINT MAARTEN (DUTCH PART)", SX, SXM, 534) >> DEFINE_COUNTRY_CODE ("SAINT MARTIN (FRENCH PART)", MF, MAF, 663) >> >> >> Query:- >> 1- > looks some countries info are missing Like “KOSOVA” ? > >Maybe we don’t need an entry for Kosovo if we don’t have a locale for >Kosovo anyway at the moment. > Agree, Since no locale for “KOSOVA” currently, So we can Ignore this case Regarding current changes, They are related with consistency Akhilesh
AKHILESH KUMAR <akhilesh.k@samsung.com> wrote: >>>> > DEFINE_COUNTRY_CODE ("SWITZERLAND", CH, CHE, 756) >>>> > DEFINE_COUNTRY_CODE ("SYRIAN ARAB REPUBLIC", SY, SYR, 760) >>>> > -DEFINE_COUNTRY_CODE ("TAIWAN, PROVINCE OF CHINA", TW, TWN, 158) >>>> > +DEFINE_COUNTRY_CODE ("TAIWAN (PROVINCE OF CHINA)", TW, TWN, 158) >>>> >>>> I don’t really see why that is an improvement. >>> >>> >>> This patch only make consistency >>> Like >>> DEFINE_COUNTRY_CODE ("SINT MAARTEN (DUTCH PART)", SX, SXM, 534) >>> DEFINE_COUNTRY_CODE ("SAINT MARTIN (FRENCH PART)", MF, MAF, 663) >>> >>> >>> Query:- >>> 1- > looks some countries info are missing Like “KOSOVA” ? >> >>Maybe we don’t need an entry for Kosovo if we don’t have a locale for >>Kosovo anyway at the moment. >> > > Agree, Since no locale for “KOSOVA” currently, So we can Ignore this case > Regarding current changes, They are related with consistency I don’t get the consistency argument here, the current text already looks fine to me.
diff --git a/locale/iso-3166.def b/locale/iso-3166.def index 52997b4..653a038 100644 --- a/locale/iso-3166.def +++ b/locale/iso-3166.def @@ -224,7 +224,7 @@ DEFINE_COUNTRY_CODE ("SWAZILAND", SZ, SWZ, 748) DEFINE_COUNTRY_CODE ("SWEDEN", SE, SWE, 752) DEFINE_COUNTRY_CODE ("SWITZERLAND", CH, CHE, 756) DEFINE_COUNTRY_CODE ("SYRIAN ARAB REPUBLIC", SY, SYR, 760) -DEFINE_COUNTRY_CODE ("TAIWAN, PROVINCE OF CHINA", TW, TWN, 158) +DEFINE_COUNTRY_CODE ("TAIWAN (PROVINCE OF CHINA)", TW, TWN, 158) DEFINE_COUNTRY_CODE ("TAJIKISTAN", TJ, TJK, 762) DEFINE_COUNTRY_CODE ("TANZANIA, UNITED REPUBLIC OF", TZ, TZA, 834) DEFINE_COUNTRY_CODE ("THAILAND", TH, THA, 764)