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

String#bytespliceの説明をruby3.3に合わせて更新 #2881

Merged
merged 6 commits into from
May 10, 2024

Conversation

hiropk
Copy link
Contributor

@hiropk hiropk commented May 9, 2024

Ruby 3.3で追加されたString#bytespliceの新しい引数について説明を追加しました。

動作確認

以下のように動作することを確認しました。

Ruby 3.2
スクリーンショット 2024-05-09 16 39 27

Ruby 3.3
スクリーンショット 2024-05-09 16 38 58

参考

https://docs.ruby-lang.org/en/3.3/String.html#method-i-bytesplice


self の一部または全部を str で置き換えて str を返します。
#@since 3.3
str_index と str_length もしくは str_range が与えられたとき、self の一部または全部を str.byteslice(str_index, str_length) もしくは str.byteslice(str_range) で置き換えて str を返します。
Copy link

@shugo shugo May 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

置き換えて str を返します。 はselfは返すの間違い(3.2.0ではstrを返していましたが、3.2.1で仕様バグ扱いでselfに変えた)ですね。
その前の行の「self の一部または全部を str で置き換えて str を返します。」の部分を「self を返します」に修正して、ここは「置き換えて str を返します。」をたんに「置き換えます」とするのがよいと思います。

@hiropk hiropk requested a review from shugo May 10, 2024 07:07
@hiropk hiropk force-pushed the update_bytesplice_docs branch from 4895a83 to 1af6b81 Compare May 10, 2024 07:24
@hiropk
Copy link
Contributor Author

hiropk commented May 10, 2024

置き換えて str を返します。 はselfは返すの間違い(3.2.0ではstrを返していましたが、3.2.1で仕様バグ扱いでselfに変えた)ですね。
その前の行の「self の一部または全部を str で置き換えて str を返します。」の部分を「self を返します」に修正して、ここは「置き換えて str を返します。」をたんに「置き換えます」とするのがよいと思います。

上記コメントを受けて、str ではなく selfを返す点について修正のコミットを積ませていただきました。
1af6b81

@hiropk hiropk force-pushed the update_bytesplice_docs branch from 1af6b81 to d939284 Compare May 10, 2024 07:37
@hiropk hiropk force-pushed the update_bytesplice_docs branch from d939284 to 6366cec Compare May 10, 2024 07:40
Copy link

@shugo shugo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hiropk ありがとうございます。修正を確認しました。

@znz 問題なければマージいただけるとありがたいです。

@znz znz merged commit d369d2d into rurema:master May 10, 2024
2 of 8 checks passed
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

Successfully merging this pull request may close these issues.

3 participants