MIDI2.0Workbench icon indicating copy to clipboard operation
MIDI2.0Workbench copied to clipboard

Is the reserved area handled correctly when generating a MUID?

Open RyoSusami opened this issue 1 year ago • 1 comments

sendDiscovery() in /libs/midici.js this._muid = getRandomInt(0xFFFFF00);

getRandomInt() in /libs/utils.js return Math.floor(Math.random() * Math.floor(max+1));

Does this mean the generated value will include 0x0FFFFF00 ?

On the other hand, the specifications... https://amei.or.jp/midistandardcommittee/MIDI2.0/MIDI2.0-DOCS/M2-101-UM_v1-2_MIDI-CI_Specification.pdf 3.3 MIDI-CI MUID The values 0x0FFFFF00 to 0x0FFFFFFE are reserved.

0x0FFFFF00 looks like it should not be used.

Please ignore if I misunderstood. Thank you.

RyoSusami avatar Sep 12 '24 08:09 RyoSusami

@RyoSusami yes you are correct. It should be getRandomInt(0xFFFFEFF) <-- One less than what it was.
Granted the chances of it ever getting 0xFFFFF00 is very very very rare :)

I'll add a fix in the next release - thanks!

starfishmod avatar Sep 15 '24 23:09 starfishmod