Project

General

Profile

Misc #11770

[PATCH] $KCODEへの代入時にwarningが発生することのテストを追加

Added by yui-knk (Kaneko Yuichiro) almost 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
[ruby-dev:49398]

Description

testケースがなかったので、追加します。


Files

0001-Add-test-cases-for-KCODE-warning.patch (749 Bytes) 0001-Add-test-cases-for-KCODE-warning.patch yui-knk (Kaneko Yuichiro), 12/04/2015 02:10 AM
0001-Add-test-cases-for-KCODE-and-warning.patch (1005 Bytes) 0001-Add-test-cases-for-KCODE-and-warning.patch yui-knk (Kaneko Yuichiro), 12/05/2015 01:09 PM

Associated revisions

Revision c0c73379
Added by yui-knk (Kaneko Yuichiro) almost 4 years ago

  • test/ruby/test_regexp.rb: Add test cases for $KCODE and $= warning [Misc #11770][ruby-dev:49398]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@53056 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 53056
Added by yui-knk (Kaneko Yuichiro) almost 4 years ago

  • test/ruby/test_regexp.rb: Add test cases for $KCODE and $= warning [Misc #11770][ruby-dev:49398]

Revision 53056
Added by yui-knk (Kaneko Yuichiro) almost 4 years ago

  • test/ruby/test_regexp.rb: Add test cases for $KCODE and $= warning [Misc #11770][ruby-dev:49398]

Revision 53056
Added by yui-knk (Kaneko Yuichiro) almost 4 years ago

  • test/ruby/test_regexp.rb: Add test cases for $KCODE and $= warning [Misc #11770][ruby-dev:49398]

Revision 53056
Added by yui-knk (Kaneko Yuichiro) almost 4 years ago

  • test/ruby/test_regexp.rb: Add test cases for $KCODE and $= warning [Misc #11770][ruby-dev:49398]

Revision 53056
Added by yui-knk (Kaneko Yuichiro) almost 4 years ago

  • test/ruby/test_regexp.rb: Add test cases for $KCODE and $= warning [Misc #11770][ruby-dev:49398]

History

Updated by nobu (Nobuyoshi Nakada) almost 4 years ago

どちらかと言えば、将来的には$KCODEの特別扱いをなしにしたい気持ちがあるので、ちょっと微妙な印象です。
それと、$=$KCODEとは別です。

Updated by yui-knk (Kaneko Yuichiro) almost 4 years ago

将来的には$KCODEの特別扱いをなしにしたい

$KCODEを削除するという意味でしょうか?
現時点ではwarningがでることをテストしておいてもいいのかなと思っています。

$=は$KCODEとは別です。

誤解していました、ありがとうございます。
test_KCODE_warningtest_ignorecase_warningに分割します。
また細かい修正になりますが、以下のテストの3・4番目のasserttionをtest_ignorecaseに切り出したほうがいいですね。

  def test_KCODE
    assert_nil($KCODE)
    assert_nothing_raised { $KCODE = nil }
    assert_equal(false, $=)
    assert_nothing_raised { $= = nil }
  end

Updated by sorah (Sorah Fukumori) almost 4 years ago

  • Assignee set to yui-knk (Kaneko Yuichiro)

Updated by naruse (Yui NARUSE) almost 4 years ago

ご自分でコミットなさるのがよいのではないでしょうか。

よいですよね>matz
redmineのruby-coreグループには追加したので、その他の手続きお願いします>しばたさん

Updated by yui-knk (Kaneko Yuichiro) almost 4 years ago

ありがとうございます:)

Updated by yui-knk (Kaneko Yuichiro) almost 4 years ago

  • Status changed from Open to Closed

Also available in: Atom PDF