CC111を使った場合のループの終了タイミングを最後のEventに変更

[At-sushi]

変更点

• CC111を使用している場合、再生終了コールバックの発生するタイミングを最後のイベントを再生した直後に変更。

RPGツクールの仕様ではend of truckメッセージまで再生する仕様になっているらしく、最後のEventまで再生しないとループのタイミングがおかしくなることがある。

[makkii-bcr]

@At-sushi
プルリクエストありがとうございます。

変更点について問題ないと思うのでマージするつもりです。
しかし、ループタイミングがおかしくなるMidiファイルで動作確認をしておきたいのですが、まだそのようなMidiファイルを探し出せていません。

もしそのようなMidiファイルをご存じであれば、Midiファイル(例えばMidi素材やゲームに含まれているMidiファイル等)のURLを教えていただけると助かります。

[At-sushi]

@makkii-bcr はじめまして。ご連絡いただきありがとうございます。

RPGツクール 2003 RTPというものに同梱されているMidiファイルにて動作確認可能です。（exe形式のインストーラなので中身を展開する必要があります）

RTP（ランタイムパッケージ）- ツクールweb

Musicフォルダ下にある「2003妖精の森.mid」というファイルが違いが分かり易いと思います。
申し訳ないですが、単体のMidiファイルで動作確認に使えるものは私も見つけられていません…

[makkii-bcr]

@At-sushi
ありがとうございます。おかげさまで動作確認ができました。

教えていただいたMidiファイルで動作確認をしました。
実は、playData.songLengthやgetTime(Number.MAX_SAFE_INTEGER)、getTiming(Number.MAX_SAFE_INTEGER)などで曲の長さを取得できるのですが、長さが少し短く取得されてしまいます。（Picotuneでは曲の終わる直前でピアノロールが一瞬止まる現象が発生しました）

変更後のparse-event.jsの
353行目 if (this.settings.isSkipEnding) songLength = lastNoteOffTiming;
354行目 tempoTrack.push({ timing:songLength, time:(60 / tempo / header.resolution) * (songLength - tempoCurTick) + tempoCurTime, value:120 });
の間に下記一行だけ追加をお願いしたいです。
if (this.settings.isCC111 && cc111Time != -1) songLength = lastEventTiming;

[At-sushi]

@makkii-bcr 該当部分修正いたしました。
play.jsのfinishTimeの部分（56行目、71-73行目）も修正したほうがいいかとも思ったのですが、ひとまずそのままにしています。

[makkii-bcr]

@At-sushi 修正ありがとうございます。
play.jsのfinishTimeの部分、分岐でどちらになっても同じ結果になるので分岐がいらなくなりますね。（this.getTime()側の代入のみでよくなる）
気づいていませんでした。
こちらも修正していただけると嬉しいです。

[At-sushi]

@makkii-bcr 修正いたしました。
この変更で set-data.js への追加分が必要なくなるので、こちらも必要であればrevertいたします。

[makkii-bcr]

@At-sushi ありがとうございます。
確かに set-data.js の追加分はなくてもよさそうですね。
ただその上のdata.～NoteOffTimeやdata.～NoteOnTimingなども追加しちゃってたりするので、
ここはこのままでもよさそうです。

[cagpie]

@At-sushi @makkii-bcr ありがとうございます！マージします

[cagpie]

v1.1.1として更新しました
https://www.npmjs.com/package/picoaudio/v/1.1.1
https://github.com/cagpie/PicoAudio.js/releases/tag/v1.1.1