Update javadocs, move to better holder

Author: lynxplay

paper-api/src/main/java/org/bukkit/MusicInstrument.java

@@ -60,9 +60,9 @@ private static MusicInstrument getInstrument(final String key) {
     }
 
     /**
-     * Gets for how long the use duration is for the instrument.
+     * Gets the use duration of this music instrument.
      *
-     * @return the duration.
+     * @return the duration expressed in seconds.
      */
     public abstract float getDuration();
 
@@ -83,7 +83,7 @@ private static MusicInstrument getInstrument(final String key) {
     /**
      * Gets the sound for this instrument.
      *
-     * @return a sound
+     * @return the sound
      */
     public abstract Sound getSound();
 

paper-server/src/main/java/org/bukkit/craftbukkit/CraftSound.java

@@ -15,7 +15,7 @@ public static Sound minecraftToBukkit(SoundEvent minecraft) {
     }
 
     public static Sound minecraftHolderToBukkit(Holder<SoundEvent> minecraft) {
-        return minecraftToBukkit(minecraft.value());
+        return CraftRegistry.minecraftHolderToBukkit(minecraft, Registries.SOUND_EVENT);
     }
 
     public static SoundEvent bukkitToMinecraft(Sound bukkit) {