ruby-changes:43570
From: duerst <ko1@a...>
Date: Tue, 12 Jul 2016 21:59:52 +0900 (JST)
Subject: [ruby-changes:43570] duerst:r55643 (trunk): * revert r55642 (previous commit) because of test failure at
duerst 2016-07-12 21:59:46 +0900 (Tue, 12 Jul 2016) New Revision: 55643 https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=55643 Log: * revert r55642 (previous commit) because of test failure at https://travis-ci.org/ruby/ruby/builds/144148780 Modified files: trunk/ChangeLog trunk/enc/iso_8859_3.c trunk/test/ruby/enc/test_case_comprehensive.rb Index: test/ruby/enc/test_case_comprehensive.rb =================================================================== --- test/ruby/enc/test_case_comprehensive.rb (revision 55642) +++ test/ruby/enc/test_case_comprehensive.rb (revision 55643) @@ -168,15 +168,8 @@ class TestComprehensiveCaseFold https://github.com/ruby/ruby/blob/trunk/test/ruby/enc/test_case_comprehensive.rb#L168 begin target = "#{test.first_data[code]}#{test.follow_data[code]*4}".encode(encoding) rescue Encoding::UndefinedConversionError - 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 + raise if code =~ /i|I/ # special case for Turkic + target = source end result = source.send(test.method_name, *test.attributes) assert_equal target, result, @@ -251,7 +244,7 @@ class TestComprehensiveCaseFold https://github.com/ruby/ruby/blob/trunk/test/ruby/enc/test_case_comprehensive.rb#L244 end generate_ascii_only_case_mapping_tests 'ISO-8859-2' - generate_case_mapping_tests 'ISO-8859-3' + generate_ascii_only_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' Index: ChangeLog =================================================================== --- ChangeLog (revision 55642) +++ ChangeLog (revision 55643) @@ -1,3 +1,8 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Tue Jul 12 21:59:40 2016 Martin Duerst <duerst@i...> + + * 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 <duerst@i...> * enc/iso_8859_3.c, test/ruby/enc/test_case_comprehensive.rb: Index: enc/iso_8859_3.c =================================================================== --- enc/iso_8859_3.c (revision 55642) +++ enc/iso_8859_3.c (revision 55643) @@ -56,7 +56,7 @@ static const UChar EncISO_8859_3_ToLower https://github.com/ruby/ruby/blob/trunk/enc/iso_8859_3.c#L56 '\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', '\151', '\272', '\273', '\274', '\255', '\256', '\277', + '\250', '\271', '\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', @@ -220,64 +220,6 @@ get_case_fold_codes_by_str(OnigCaseFoldT https://github.com/ruby/ruby/blob/trunk/enc/iso_8859_3.c#L220 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<end && to<to_end) { - code = *(*pp)++; - if (code==SHARP_s) { - if (flags&ONIGENC_CASE_UPCASE) { - flags |= ONIGENC_CASE_MODIFIED; - *to++ = 'S'; - code = (flags&ONIGENC_CASE_TITLECASE) ? 's' : 'S'; - } - else if (flags&ONIGENC_CASE_FOLD) { - flags |= ONIGENC_CASE_MODIFIED; - *to++ = 's'; - code = 's'; - } - } - else if (code==0xB5) ; - else if ((EncISO_8859_3_CtypeTable[code] & BIT_CTYPE_UPPER) - && (flags & (ONIGENC_CASE_DOWNCASE|ONIGENC_CASE_FOLD))) { - flags |= ONIGENC_CASE_MODIFIED; - if (code=='I') - code = flags&ONIGENC_CASE_FOLD_TURKISH_AZERI ? DOTLESS_i : 'i'; - else - code = ENC_ISO_8859_3_TO_LOWER_CASE(code); - } - else if ((EncISO_8859_3_CtypeTable[code]&BIT_CTYPE_LOWER) - && (flags&ONIGENC_CASE_UPCASE)) { - flags |= ONIGENC_CASE_MODIFIED; - if (code=='i') - code = flags&ONIGENC_CASE_FOLD_TURKISH_AZERI ? I_WITH_DOT_ABOVE : 'I'; - else if (code==DOTLESS_i) - code = 'I'; - else if (code>=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 */ @@ -298,7 +240,7 @@ OnigEncodingDefine(iso_8859_3, ISO_8859_ https://github.com/ruby/ruby/blob/trunk/enc/iso_8859_3.c#L240 0, ONIGENC_FLAG_NONE, #ifdef ONIG_CASE_MAPPING - case_map, + onigenc_single_byte_ascii_only_case_map, #endif /* ONIG_CASE_MAPPING */ }; ENC_ALIAS("ISO8859-3", "ISO-8859-3") -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/