PacketUtils comment

src/main/java/net/minestom/server/network/netty/NettyServer.java

@@ -52,9 +52,9 @@ public class NettyServer {
             channel = NioServerSocketChannel.class;
         }
 
-        bootstrap = new ServerBootstrap();
-        bootstrap.group(boss, worker);
-        bootstrap.channel(channel);
+        bootstrap = new ServerBootstrap()
+                .group(boss, worker)
+                .channel(channel);
 
         bootstrap.childHandler(new ChannelInitializer<SocketChannel>() {
             protected void initChannel(@NotNull SocketChannel ch) {

src/main/java/net/minestom/server/utils/PacketUtils.java

@@ -10,7 +10,8 @@ import org.jetbrains.annotations.NotNull;
 import java.util.Collection;
 
 /**
- * Class used to write packets.
+ * Utils class for packets. Including writing a {@link ServerPacket} into a {@link ByteBuf}
+ * for network processing.
  */
 public final class PacketUtils {
 
@@ -18,7 +19,15 @@ public final class PacketUtils {
 
     }
 
-    public static void sendGroupedPacket(Collection<Player> players, ServerPacket packet) {
+    /**
+     * Sends a {@link ServerPacket} to multiple players. Mostly used for convenience.
+     * <p>
+     * Be aware that this will cause the send packet listeners to be given the exact same packet object.
+     *
+     * @param players the players to send the packet to
+     * @param packet  the packet to send to the players
+     */
+    public static void sendGroupedPacket(@NotNull Collection<Player> players, @NotNull ServerPacket packet) {
         for (Player player : players) {
             player.getPlayerConnection().sendPacket(packet);
         }
@@ -31,7 +40,6 @@ public final class PacketUtils {
      * @param packet the packet to write into {@code buf}
      */
     public static void writePacket(@NotNull ByteBuf buf, @NotNull ServerPacket packet) {
-
         final ByteBuf packetBuffer = getPacketBuffer(packet);
 
         writePacket(buf, packetBuffer, packet.getId());

src/main/java/net/minestom/server/utils/binary/BinaryReader.java

@@ -11,6 +11,7 @@ import net.minestom.server.utils.NBTUtils;
 import net.minestom.server.utils.SerializerUtils;
 import net.minestom.server.utils.Utils;
 import net.minestom.server.utils.validate.Check;
+import org.jetbrains.annotations.NotNull;
 import org.jglrxavpok.hephaistos.nbt.NBT;
 import org.jglrxavpok.hephaistos.nbt.NBTException;
 import org.jglrxavpok.hephaistos.nbt.NBTReader;
@@ -29,7 +30,7 @@ public class BinaryReader extends InputStream {
     private final ByteBuf buffer;
     private final NBTReader nbtReader = new NBTReader(this, false);
 
-    public BinaryReader(ByteBuf buffer) {
+    public BinaryReader(@NotNull ByteBuf buffer) {
         this.buffer = buffer;
     }
 

src/main/java/net/minestom/server/utils/binary/BinaryWriter.java

@@ -26,7 +26,7 @@ public class BinaryWriter extends OutputStream {
     private final NBTWriter nbtWriter = new NBTWriter(this, false);
 
     /**
-     * Creates a {@link BinaryWriter} with a custom initial capacity.
+     * Creates a {@link BinaryWriter} using a heap buffer with a custom initial capacity.
      *
      * @param initialCapacity the initial capacity of the binary writer
      */