Bug #591

io.c の rb_io_extract_modeenc で変数 has_vmode は不用か?

Added by Martin Dürst almost 7 years ago. Updated about 4 years ago.

Status:Closed
Priority:Normal
Assignee:Yukihiro Matsumoto
ruby -v: Backport:

Description

=begin
io.c の rb_io_extract_modeenc で変数 has_vmode は 0 に初期化され、
最初に if (!has_vmode) { (必ず真) でテストされ、後に 1 に
設定され、その後全然使われてないので、不用のではないかと思います。
has_enc は実際に使われているが、has_vperm はそもぞもないので、
has_vmode はその間の中途半端なものになってしまったみたいです。
(調べましたら関連の三行はいずれもまつもとさんが praise されましたので、
担当者をまつもとさんにしましたが、こちらで取り除いてもいいです。)
=end

History

#1 Updated by Martin Dürst almost 7 years ago

  • Status changed from Open to Closed

=begin
ごめんなさい。こちらの勘違いでした。goto まで読めなかったです。
=end

#2 Updated by Yukihiro Matsumoto almost 7 years ago

=begin
まつもと ゆきひろです

In message "Re: Bug #591 io.c の rb_io_extract_modeenc で変数 has_vmode は不用か?"
on Wed, 24 Sep 2008 17:06:05 +0900, Martin Dürst redmine@ruby-lang.org writes:

|ごめんなさい。こちらの勘違いでした。goto まで読めなかったです。

変なコードを書くのがいけないと怒られそうなところです。

=end

Also available in: Atom PDF