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

Add GC.verify_compaction_references since 2.7.0 #2116

Merged
merged 2 commits into from Mar 10, 2020

Conversation

@znz
Copy link
Member

znz commented Jan 27, 2020

No description provided.

@znz znz added the Ruby2.7.0 label Jan 27, 2020

コンパクションの参照の一貫性を検証します。

このメソッドは実装の詳細です。

This comment has been minimized.

Copy link
@pocke

pocke Jan 28, 2020

Member

このメソッドは実装の詳細です。

はRDocの「This method is implementation specific.」の訳だと思うのですが、パッと読んで意味がわかりませんでした。

https://github.com/ruby/ruby/blob/79f0ed3529c096c1182ccfeb7a4ad316455fc90e/gc.c#L8650-L8665

推測するに、「このメソッドはCRubyの実装に特有のメソッドです」みたいなことが原文では言いたいのかなと思いました。

This comment has been minimized.

Copy link
@znz

znz Jan 28, 2020

Author Member

ユーザーが直接使うのは GC.compact の方で、 GC.verify_compaction_references は後悔しているけど、あまり直接使うものではない、という意味かと思ったのですが、こちらにだけ rdoc が書いてあって GC.compact には書いていないので、実際の意図はわかりませんでした。

This comment has been minimized.

Copy link
@pocke

pocke Jan 28, 2020

Member

参考までに、 ObjectSpace.count_objectsの戻り値に関しても"implementation specific"という表現が使われていて、そこから意味を推測しました。

https://github.com/ruby/ruby/blob/79f0ed3529c096c1182ccfeb7a4ad316455fc90e/gc.c#L4016-L4049

GC.verify_internal_consistencyにも同じ表現が使われているけど、こっちは知らないメソッドで分からなかった…

This comment has been minimized.

Copy link
@znz

znz Mar 10, 2020

Author Member

時間をおいて読み直してみると、そちらの解釈の方があっていそうだったので、更新しました。

@hanachin hanachin merged commit bc45c33 into rurema:master Mar 10, 2020
3 checks passed
3 checks passed
rake
Details
misspell
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.