-
Notifications
You must be signed in to change notification settings - Fork 328
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
Conversation
refm/api/src/_builtin/String
Outdated
|
||
self の一部または全部を str で置き換えて str を返します。 | ||
#@since 3.3 | ||
str_index と str_length もしくは str_range が与えられたとき、self の一部または全部を str.byteslice(str_index, str_length) もしくは str.byteslice(str_range) で置き換えて str を返します。 |
There was a problem hiding this comment.
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 を返します。」をたんに「置き換えます」とするのがよいと思います。
4895a83
to
1af6b81
Compare
上記コメントを受けて、str ではなく selfを返す点について修正のコミットを積ませていただきました。 |
1af6b81
to
d939284
Compare
d939284
to
6366cec
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ruby 3.3で追加されたString#bytespliceの新しい引数について説明を追加しました。
動作確認
以下のように動作することを確認しました。
Ruby 3.2
Ruby 3.3
参考
https://docs.ruby-lang.org/en/3.3/String.html#method-i-bytesplice