feat: support trumpets in nbs v5 format

This commit is contained in:
amy 2026-03-29 15:39:30 -03:00
parent 624d87c381
commit 84df266240
No known key found for this signature in database
2 changed files with 32 additions and 20 deletions

View file

@ -30,24 +30,20 @@ public class Instrument {
public final int offset;
public final String sound;
private Instrument (final int id, final String name, final int offset, final String sound) {
protected Instrument (final int id, final String name, final int offset, final String sound) {
this.id = id;
this.name = name;
this.offset = offset;
this.sound = sound;
}
private Instrument (final int id, final String name, final int offset) {
protected Instrument (final int id, final String name, final int offset) {
this.id = id;
this.name = name;
this.offset = offset;
this.sound = "block.note_block." + name;
}
public static Instrument of (final String sound) {
return new Instrument(-1, null, 0, sound);
}
public static Instrument fromId (final int id) {
return VALUES[id];
}

View file

@ -4,6 +4,9 @@ import java.io.IOException;
import java.nio.ByteBuffer;
import java.nio.ByteOrder;
import java.util.ArrayList;
import net.minecraft.resources.Identifier;
import net.minecraft.util.Util;
import net.minecraft.world.phys.Vec3;
public class NBSConverter {
@ -120,12 +123,12 @@ public class NBSConverter {
if (buffer.hasRemaining()) {
final byte customInstrumentCount = buffer.get();
for (int i = 0; i < customInstrumentCount; i++) {
final NBSCustomInstrument customInstrument = new NBSCustomInstrument();
customInstrument.name = getString(buffer, bytes.length);
customInstrument.file = getString(buffer, bytes.length);
customInstrument.pitch = buffer.get();
customInstrument.key = buffer.get() != 0;
customInstruments.add(customInstrument);
final String name = getString(buffer, bytes.length);
final String file = getString(buffer, bytes.length);
final byte pitch = buffer.get();
final boolean visible = buffer.get() != 0; // "Press piano key"
customInstruments.add(new NBSCustomInstrument(name, file, pitch, visible));
}
}
@ -145,12 +148,10 @@ public class NBSConverter {
key = (double) ((note.key * 100) + note.pitch) / 100;
} else {
final int index = note.instrument - vanillaInstrumentCount;
if (index >= customInstruments.size()) continue;
final NBSCustomInstrument customInstrument = customInstruments.get(index);
instrument = Instrument.of(customInstrument.name);
instrument = customInstrument;
key = (note.key) + (customInstrument.pitch + (double) note.pitch / 100);
}
@ -220,10 +221,25 @@ public class NBSConverter {
public byte stereo = 100;
}
private static class NBSCustomInstrument {
public String name;
public String file;
public byte pitch = 0;
public boolean key = false;
private static class NBSCustomInstrument extends Instrument {
public final byte pitch;
public final boolean visible;
public NBSCustomInstrument(final String name, final String optionalPath, final byte pitch, final boolean visible) {
// some implementations straight up just don't save the path, so we perform conversions on name on those
final String path = optionalPath.isBlank() ? name : optionalPath;
String sound = Util.sanitizeName(path, Identifier::isAllowedInIdentifier)
.replaceFirst("\\.ogg$", "")
.replaceFirst("^minecraft/", "")
.replace('/', '.');
// if there's no path, it's probably a note block sound
// this happens when you have trumpets saved in a v5 format NBS for example
if (!sound.contains(".")) sound = "block.note_block." + sound;
super(-1, name, 0, sound);
this.pitch = pitch;
this.visible = visible;
}
}
}