Issue #7814 has been updated by yhara (Yutaka HARA).


遅くなりましたがArray#bsearchのレビューを行いました。

* 配列のソートは降順でも問題ない、というので合ってると思います。
* 「求める値がブロックパラメータの値よりも大きい（0 <= k < i）場合」→ find-anyモードはある範囲の値を探すので、「求める値の範囲が〜」と書いたほうがいいかなと思いました。
* 最後の「It is undefined which value is actually picked up at each iteration.」はモード混在の話とは関係ない説明かと思います。

ということで修正を r5763 にコミットしました。(refsつけ忘れました。すいません)
Range#bsearchの方も合わせて直しておきました。(もとのrdocにtypoがある気がしたので、
http://bugs.ruby-lang.org/issues/8242 として報告しました)
----------------------------------------
Bug #7814: 2.0 対応 _builtin/A* 〜 _builtin/L* のレビュー

https://bugs.ruby-lang.org/issues/7814#change-38396

Author: sho-h (Sho Hashimoto)
Status: Open
Priority: Normal
Assignee: sho-h (Sho Hashimoto)
Category: doc:review
Target version: 
reporter: 
ruby_version: 


=begin
変則的ですが 2.0 対応の _builtin/A* 〜 _builtin/L* のレビューをお願いします。

やり方は http://redmine.ruby-lang.org/wiki/rurema/Phase3ReviewProcess1 を参照してください。

以下、対応したものの一覧です。Array#bsearch と Kernel.#caller_locations についてはあまり自信がありません。(2/9 現在は間違ってるように感じていて後で自分でも見直そうかと思います)

* ARGF#inspect の追加
* ARGF#bytes, chars, #lines の警告の追加
* Array#bsearch の追加
  * 「昇順にソート」という翻訳は正しい?
  * 「find-any モード」の条件は正しい?
* ENV.to_h の追加
* File.fnmatch? で File::FNM_EXTGLOB が指定できるようになった記述の追記
* File::FNM_EXTGLOB の追加
* Fiber#resume で #transfer の呼び出し後は例外になる記述の追記
* GC::Profiler.raw_data の追加
* Hash#to_h の追加
* NilClass#to_h、Struct#to_h の追加
  * Struct は他にも #inspect、#to_s、#select がなかったので追加(本チケットとは無関係)
* Hash#default_proc= に nil が指定できる記述の追記
* IO#lines, #bytes, #chars and #codepoints
  * ※ バージョンの分岐が大変な事になってるので読み辛い
* Kernel.#Hash の追加
* Kernel.#__dir__ の追加
* Kernel.#caller_locations の追加
  * 用語の使い方は正しい？
* Kernel.#warn の引数の修正

* Kernel.#caller の引数の修正

* LoadError#path の追加
* Object#respond_to? の第二引数の記述の修正

* Object.#to_enum, .#enum_for にブロックを指定した場合の記述の追記

以下、NEWS には記載されていますが、細部の話であるため需要が挙がってから対応の方がよいと考えて省略したものの一覧です。

* Array#shuffle!、Array#sample
* Array#values_at
* Kernel.#system, .#exec
* Kernel.#__callee__
* Kernel.#inspect
=end


-- 
http://bugs.ruby-lang.org/

--
ML: ruby-reference-manual@m...
Info: http://QuickML.com/