Project

General

Profile

Bug #1433

test_sprintf_p fails

Added by naruse (Yui NARUSE) over 10 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Target version:
ruby -v:
ruby 1.9.2dev (2009-05-02 trunk 23326) [x86_64-freebsd7.2]
Backport:
[ruby-dev:38374]

Description

=begin
test_sprintf_p(TestM17N) [/home/naruse/git/ruby/test/ruby/test_m17n.rb:773]:
<#Encoding:US-ASCII> expected but was
<#Encoding:ASCII-8BIT>.

1.9.1のパッチリリースのblocking bugである本件ですが、
これはテスト側を動作に合わせるべきだと思っています。

この部分を最後に修正したのはうささんのようですが、どう思われますか。
あと他の方も。
=end

History

#1

Updated by usa (Usaku NAKAMURA) over 10 years ago

=begin
こんにちは、なかむら(う)です。

In message "[ruby-dev:38374] [Bug #1433] test_sprintf_p fails"
on May.05,2009 09:52:14, redmine@ruby-lang.org wrote:

この部分を最後に修正したのはうささんのようですが、どう思われますか。
あと他の方も。

たぶんr15117に追従したテストの変更だと思いますが、当時の仕様
としては正しかったのでしょう。
なんで幅指定があるときだけUS-ASCIIになるべきだったのかは今考
えても思い出せないのですが(なかださんわかりますか?)、当時と現
状で動作が異なるということは、現状の動作にした方に説明してい
ただくのがよいかと思います。誰のどの変更だろう?

それでは。
--
U.Nakamura usa@garbagecollect.jp

=end

#2

Updated by naruse (Yui NARUSE) over 10 years ago

=begin
成瀬です。

(2009/05/07 11:48), U.Nakamura さんは書きました:

こんにちは、なかむら(う)です。

In message "[ruby-dev:38374] [Bug #1433] test_sprintf_p fails"
on May.05,2009 09:52:14, redmine@ruby-lang.org wrote:

この部分を最後に修正したのはうささんのようですが、どう思われますか。
あと他の方も。

たぶんr15117に追従したテストの変更だと思いますが、当時の仕様
としては正しかったのでしょう。
なんで幅指定があるときだけUS-ASCIIになるべきだったのかは今考
えても思い出せないのですが(なかださんわかりますか?)、当時と現
状で動作が異なるということは、現状の動作にした方に説明してい
ただくのがよいかと思います。誰のどの変更だろう?

r23186とr23187のわたしの変更なようです。
当時どうしたんだっけ、テストでエラーがでるけどこれはいっかと思ったような、
とりあえずテストはおいておくかと思ったような。

エンコーディングの変更は意図したものではないんですが、
結果的には他との整合性を考えても、元のエンコーディングを保持するという
現状の動作が正しいのではないかと思います。

当時は元のエンコーディングに関係なく、7bitならばUS-ASCIIというルールが
あったので、
そのためにUS-ASCIIになっていたのですが、それはその後変更されており、
このテストでも当該assert以外はASCII-8BITに変更されています。
US-ASCIIになるルールが廃止された時点でこのテストと、
これに対応する挙動が誤っていたという感じですかね。

--
NARUSE, Yui

=end

#3

Updated by yugui (Yuki Sonoda) about 10 years ago

  • Status changed from Assigned to Closed

=begin

=end

Also available in: Atom PDF