From 19b5e818dd7191bd4a04d57582aa92e774f3f053 Mon Sep 17 00:00:00 2001 From: duerst Date: Wed, 13 Jul 2016 00:02:34 +0000 Subject: [PATCH] * enc/iso_8859_3.c, test/ruby/enc/test_case_comprehensive.rb: Implement non-ASCII case conversion for ISO-8859-3, by Takuya Miyamoto. * test/ruby/enc/test_case_comprehensive.rb: Extend special treatment for Turkic. * enc/iso_8859_3.c: Exclude dotless i/I with dot from case-insensitive matching because they are not a case pair. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@55648 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 30 ++++++----- enc/iso_8859_3.c | 64 ++++++++++++++++++++++-- test/ruby/enc/test_case_comprehensive.rb | 13 +++-- 3 files changed, 85 insertions(+), 22 deletions(-) diff --git a/ChangeLog b/ChangeLog index 1778f8076a..768c15e707 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,19 +1,4 @@ -Wed Jul 13 08:40:21 2016 Martin Duerst - - * test/ruby/enc/test_iso_8859.rb: Excluded dotless i/I with dot from - case-insensitive matching because they are not a case pair. - -Tue Jul 12 23:13:43 2016 Nobuyoshi Nakada - - * math.c (rb_math_sqrt): [EXPERIMENTAL] move Complex sqrt support - from mathn.rb. - -Tue Jul 12 21:59:40 2016 Martin Duerst - - * revert r55642 (previous commit) because of test failure at - https://travis-ci.org/ruby/ruby/builds/144148780 - -Tue Jul 12 21:33:10 2016 Martin Duerst +Wed Jul 13 09:02:30 2016 Martin Duerst * enc/iso_8859_3.c, test/ruby/enc/test_case_comprehensive.rb: Implement non-ASCII case conversion for ISO-8859-3, by Takuya Miyamoto. @@ -21,6 +6,19 @@ Tue Jul 12 21:33:10 2016 Martin Duerst * test/ruby/enc/test_case_comprehensive.rb: Extend special treatment for Turkic. + * enc/iso_8859_3.c: Exclude dotless i/I with dot from case-insensitive + matching because they are not a case pair. + +Wed Jul 13 08:40:21 2016 Martin Duerst + + * test/ruby/enc/test_iso_8859.rb: Exclude dotless i/I with dot from + case-insensitive matching because they are not a case pair. + +Tue Jul 12 23:13:43 2016 Nobuyoshi Nakada + + * math.c (rb_math_sqrt): [EXPERIMENTAL] move Complex sqrt support + from mathn.rb. + Tue Jul 12 01:25:09 2016 NAKAMURA Usaku * configure.in, lib/mkmf.rb, win32/Makefile.sub (CSRCFLAG): make the diff --git a/enc/iso_8859_3.c b/enc/iso_8859_3.c index e6dee1042a..a37a3cf0a7 100644 --- a/enc/iso_8859_3.c +++ b/enc/iso_8859_3.c @@ -56,7 +56,7 @@ static const UChar EncISO_8859_3_ToLowerCaseTable[256] = { '\220', '\221', '\222', '\223', '\224', '\225', '\226', '\227', '\230', '\231', '\232', '\233', '\234', '\235', '\236', '\237', '\240', '\261', '\242', '\243', '\244', '\245', '\266', '\247', - '\250', '\271', '\272', '\273', '\274', '\255', '\256', '\277', + '\250', '\151', '\272', '\273', '\274', '\255', '\256', '\277', '\260', '\261', '\262', '\263', '\264', '\265', '\266', '\267', '\270', '\271', '\272', '\273', '\274', '\275', '\276', '\277', '\340', '\341', '\342', '\303', '\344', '\345', '\346', '\347', @@ -161,7 +161,7 @@ is_code_ctype(OnigCodePoint code, unsigned int ctype, OnigEncoding enc ARG_UNUSE static const OnigPairCaseFoldCodes CaseFoldMap[] = { { 0xa1, 0xb1 }, { 0xa6, 0xb6 }, - { 0xa9, 0xb9 }, + /*{ 0xa9, 0xb9 }, exclude dotless i/I with dot; not a case pair */ { 0xaa, 0xba }, { 0xab, 0xbb }, { 0xac, 0xbc }, @@ -220,6 +220,64 @@ get_case_fold_codes_by_str(OnigCaseFoldType flag, flag, p, end, items); } +#ifdef ONIG_CASE_MAPPING +#define DOTLESS_i (0xB9) +#define I_WITH_DOT_ABOVE (0xA9) +static int +case_map (OnigCaseFoldType* flagP, const OnigUChar** pp, + const OnigUChar* end, OnigUChar* to, OnigUChar* to_end, + const struct OnigEncodingTypeST* enc) +{ + OnigCodePoint code; + OnigUChar *to_start = to; + OnigCaseFoldType flags = *flagP; + + while (*pp=0xB0 && code<=0xBF ) { + code -= 0x10; + } + else { + code -= 0x20; + } + } + *to++ = code; + if (flags&ONIGENC_CASE_TITLECASE) /* switch from titlecase to lowercase for capitalize */ + flags ^= (ONIGENC_CASE_UPCASE|ONIGENC_CASE_DOWNCASE|ONIGENC_CASE_TITLECASE); + } + *flagP = flags; + return (int)(to-to_start); +} +#endif /* ONIG_CASE_MAPPING */ + OnigEncodingDefine(iso_8859_3, ISO_8859_3) = { onigenc_single_byte_mbc_enc_len, "ISO-8859-3", /* name */ @@ -240,7 +298,7 @@ OnigEncodingDefine(iso_8859_3, ISO_8859_3) = { 0, ONIGENC_FLAG_NONE, #ifdef ONIG_CASE_MAPPING - onigenc_single_byte_ascii_only_case_map, + case_map, #endif /* ONIG_CASE_MAPPING */ }; ENC_ALIAS("ISO8859-3", "ISO-8859-3") diff --git a/test/ruby/enc/test_case_comprehensive.rb b/test/ruby/enc/test_case_comprehensive.rb index 9a5c17724d..8d202ca148 100644 --- a/test/ruby/enc/test_case_comprehensive.rb +++ b/test/ruby/enc/test_case_comprehensive.rb @@ -168,8 +168,15 @@ class TestComprehensiveCaseFold begin target = "#{test.first_data[code]}#{test.follow_data[code]*4}".encode(encoding) rescue Encoding::UndefinedConversionError - raise if code =~ /i|I/ # special case for Turkic - target = source + if test.first_data[code]=="i\u0307" or test.follow_data[code]=="i\u0307" # explicit dot above + first_data = test.first_data[code]=="i\u0307" ? 'i' : test.first_data[code] + follow_data = test.follow_data[code]=="i\u0307" ? 'i' : test.follow_data[code] + target = "#{first_data}#{follow_data*4}".encode(encoding) + elsif code =~ /i|I/ # special case for Turkic + raise + else + target = source + end end result = source.send(test.method_name, *test.attributes) assert_equal target, result, @@ -244,7 +251,7 @@ class TestComprehensiveCaseFold end generate_ascii_only_case_mapping_tests 'ISO-8859-2' - generate_ascii_only_case_mapping_tests 'ISO-8859-3' + generate_case_mapping_tests 'ISO-8859-3' generate_case_mapping_tests 'ISO-8859-4' generate_ascii_only_case_mapping_tests 'ISO-8859-5' generate_ascii_only_case_mapping_tests 'ISO-8859-7'