Root: Change parameters to pass by value #345
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
45e0ec4 の修正でメンバ関数の引数を参照渡しに変更しましたが、 関数内処理の影響で引数の参照値が変化し板移転の処理に支障が 生じることが分かりました。そのため引数を値渡しに変更することで 意図しない値の更新を防ぎます。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
45e0ec4 の修正でメンバ関数の引数を参照渡しに変更しましたが、関数内処理の影響で引数の参照値が変化し板移転の処理に支障が生じることが分かりました。
そのため引数を値渡しに変更することで意図しない値の更新を防ぎます。