Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

English library で定義されるグローバル変数が網羅されていない #1165

Closed
pocke opened this issue Apr 1, 2018 · 5 comments

Comments

@pocke
Copy link
Member

pocke commented Apr 1, 2018

https://docs.ruby-lang.org/ja/latest/library/English.html に書かれている、定義されるグローバル変数が全てを網羅していないようです。

Ruby本体のドキュメントと比べると、以下のものが抜けています。
https://github.com/ruby/ruby/blob/v2_5_1/lib/English.rb#L23-L47

  • $FS
  • $NR
  • $OUTPUT_FIELD_SEPARATOR
  • $OUTPUT_RECORD_SEPARATOR
  • $PROCESS_ID
  • $RS

また、ruremaのドキュメントでは$PROGRAM_NAMEが定義されると書かれていますが、Ruby本体のドキュメントでは$PROGRAM_NAMEの記述はありません。
これは$PROGRAM_NAMEが1.9.3からビルドインの変数に変わったためのようです。
ruby/ruby@6dcbfbc

Fri Apr 2 11:36:20 2004 Minero Aoki aamine@loveruby.net

  • eval.c (Init_load): make $LOADED_FEATURES built-in.
    [ruby-dev:23299]

  • ruby.c (ruby_prog_init): make $PROGRAM_NAME built-in.

  • lib/English.rb: remove $LOADED_FEATURES and $PROGRAM_NAME.

https://github.com/ruby/ruby/blob/85883dc393f60adc3043d28899eff3f1ce53b4a5/doc/ChangeLog-1.9.3

Kernelのドキュメントには既に$PROGRAM_NAMEは存在する( https://docs.ruby-lang.org/ja/latest/class/Kernel.html#V_0 )ので、English libraryのドキュメントからは単に削除して良いように思えます。

pocke added a commit to pocke/doctree that referenced this issue Apr 8, 2018
`$LOADED_FEATURES`と`$PROGRAM_NAME`はEnglishではなくビルドインで定義されているため、English libraryからは記述を削除しました。(詳しくは rurema#1165 を参照。`$LOADED_FEATURES`に関してはIssueに記述はありませんが、`$PROGRAM_NAME`と同じコミットでEnglishから削除されています。)
また、ruremaではRuby 1.8, 1.9をサポートしていないので、不要となっている分岐を削除しました。
@sho-h
Copy link
Member

sho-h commented Apr 23, 2018

$FS$RS までエントリ自体はある($FIELD_SEPARATOR$INPUT_RECORD_SEPARATOR だけ表示されてる)のでbitclustぽいですね。

一方、インスタンスメソッドについては例えば Array#map! と #collect! だけ見ましたがどちらも表示されていますので、少なくともちぐはぐな状況ではあるようです。

どっちに寄せるかという話になるかと思うのですが、ページ内検索で探しにくいと思いますので、グローバル変数も全部表示するのがいいのではないかと思いました。

@pocke
Copy link
Member Author

pocke commented Apr 23, 2018

どっちに寄せるかという話になるかと思うのですが、ページ内検索で探しにくいと思いますので、グローバル変数も全部表示するのがいいのではないかと思いました。

そうですね、私も全部表示したほうがわかりやすいと思います。

このIssueはCloseして https://github.com/rurema/bitclust の方に新しくIssueを立てたほうが良いでしょうか??

@sho-h
Copy link
Member

sho-h commented Apr 23, 2018

このIssueはCloseして https://github.com/rurema/bitclust の方に新しくIssueを立てたほうが良いでしょうか??

あ、そうですね。ではお願いします!closeはしてしまいますね。

@pocke
Copy link
Member Author

pocke commented Apr 23, 2018

Issueたてました!

@sho-h
Copy link
Member

sho-h commented Apr 23, 2018

ありがとうございます!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants