kuma4649 / mucomdotnet Goto Github PK

古代さんより指摘。

未調査。

I tried WAV output, but it doesn't work. The same goes for VGM.
Looking at the source, I don't think VGM has been implemented.
Hasn't it been implemented yet?

Just in case, I've debugged WAV.program.cs, and there's an exception here.

drv = new Driver();
((Driver)drv).Init(
lca
, bl.ToArray()
, null
, new object[] {
false
, false
, false
}
);

Is WAV not implemented yet too?

モバイルやブラウザなど、ファイルシステムに自由にファイルの読み書きができない環境があるので、ファイルを前提にデーターの読み書きをする構造だとそのような環境で動かすことが難しくなります。

こういうケースは Stream クラスで読み書きを抽象化するのが定石だと思うので、そのような形にしていただきたいのです。
MUCOM の場合、 MML 内に記述した追加ファイルの読み込みがあるので、その辺りを考慮して疑似コードで定義案を作成しました。そちらも見ていただければと思います。

https://gist.github.com/aosoft/3c6461e827aed52057abcb1d2198c43b

古代さんより＞
AMC/AMDと同じようにポルタメントを１オクターブ以上行えるようにしたい
対応したところバグ発生。

既存のデータを少しでも互換性が高い状態で再生させたいため。

Sコマンドの4つのパラメータの範囲を8bit(0～255)から16bit(-32768～32767)に拡張すると使い勝手が良くなりそう。
・拡張した場合の懸念点：コマンドサイズの肥大化、互換性の喪失
・その他：オリジナルのMUCOM88では、音長が128クロックを超えるとSコマンドの値が他のトラックのディチューン値に影響を与えるバグが存在する

public void Init(string fileName, Action<ChipDatum> chipWriteRegister, Action<long, int> chipWaitSend, MmlDatum[] srcBuf, object addtionalOption) の
{
// 略
    pcm = GetPCMFromSrcBuf();
    if (pcm == null) GetPCMDataFromFile();

上記 2 行をコメントアウトして pcm = null の状態にし、 PCM 使用曲を読み込ませると例外で落ちます。

static な Log.writeMethod にセットしているので、 Compiler や Driver のインスタンスを同時に使用している場合に問題を起こす可能性があります。

Log.writeMethod はアクセスに制限がないので、

• StartToMmlData を呼ぶ前にセットしておいてもらうようにして StartToMmlData の引数からは削除する
• StartToMmlData の呼び出し内でのみ使用する形に限定し、 Log クラス側に影響を与えないようにする

のどちらかにすべきと思います。

msg クラスのクラスコンストラクタで固定的にメッセージが読み込まれるためアプリから読み込み処理を制御できません。
この処理で期待される配置にメッセージリソースが置けるとは限らないので読み込みは自前で行いたい。