MSP_AllOffPatchのリファクタ #1
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
elleonard
commented
Oct 20, 2020
| @@ -7,6 +7,8 @@ | |||
| * @plugindesc MultiSoundPlayerで設定したサウンドを終了時などに一括で解除します。 | |||
| * @author 沫那環(Tamaki Awana) | |||
| * | |||
| * @base MultiSoundPlayer | |||
There was a problem hiding this comment.
MZプラグインは @base を指定することにより、プラグインの依存関係を明示できます。
これを書いておくと、このプラグイン単体で導入しようとした際にMZエディタに警告が表示されます。
elleonard
commented
Oct 20, 2020
| * 指定したサウンドを削除する（元プラグインのUtility.delSoundを移植） | ||
| * @param {string|number} soundId サウンドID | ||
| */ | ||
| function deleteSound (soundId) { |
There was a problem hiding this comment.
ここだけ自分用のインデント設定で書いてしまいましたが、元々4スペースか8スペースかで統一されていないようです。
あとで、お好みのインデントに調整していただければと思います。
ちなみに、VSCodeをお使いであれば、 Shift + Alt + F （Macの場合は Shift + Option + F）で自動フォーマットをかけてくれます。
エディタの設定でTab Sizeをお好みの幅に設定してから実行してみてください。
|
すっきりと整ったコードのリファクタPRをありがとうございました。 |
|
（ミスってCloseしてしまった……！！ 滝汗） |
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.
https://tm.lucky-duet.com/viewtopic.php?f=116&t=9926 の書き込みを拝見しました。
ループ周りを整理して見通しを良くしてみましたが、いかがでしょうか。