added more nullability information

Author: david

src/main/java/net/onelitefeather/bettergopaint/BetterGoPaint.java

@@ -39,6 +39,7 @@
 import org.incendo.cloud.execution.ExecutionCoordinator;
 import org.incendo.cloud.paper.LegacyPaperCommandManager;
 import org.incendo.serverlib.ServerLib;
+import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import java.io.File;
@@ -47,8 +48,8 @@
 
 public class BetterGoPaint extends JavaPlugin implements Listener {
 
-    private final PlayerBrushManager brushManager = new PlayerBrushManager();
-    private final Metrics metrics = new Metrics(this, 18734);
+    private final @NotNull PlayerBrushManager brushManager = new PlayerBrushManager();
+    private final @NotNull Metrics metrics = new Metrics(this, 18734);
 
     @Override
     public void onLoad() {
@@ -131,7 +132,7 @@ private boolean hasOriginalGoPaint() {
         }
     }
 
-    public PlayerBrushManager getBrushManager() {
+    public @NotNull PlayerBrushManager getBrushManager() {
         return brushManager;
     }
 

src/main/java/net/onelitefeather/bettergopaint/utils/Height.java

@@ -19,14 +19,15 @@
 package net.onelitefeather.bettergopaint.utils;
 
 import org.bukkit.Location;
+import org.jetbrains.annotations.NotNull;
 
 public class Height {
 
-    public static int getHeight(Location location) {
+    public static int getHeight(@NotNull Location location) {
         return location.getWorld().getHighestBlockYAt(location);
     }
 
-    public static double getAverageHeightDiffFracture(Location location, int height, int distance) {
+    public static double getAverageHeightDiffFracture(@NotNull Location location, int height, int distance) {
         double totalHeight = 0;
         totalHeight += Math.abs(getHeight(location.clone().add(distance, 0, -distance))) - height;
         totalHeight += Math.abs(getHeight(location.clone().add(distance, 0, distance))) - height;
@@ -39,7 +40,7 @@ public static double getAverageHeightDiffFracture(Location location, int height,
         return (totalHeight / 8d) / distance;
     }
 
-    public static double getAverageHeightDiffAngle(Location location, int distance) {
+    public static double getAverageHeightDiffAngle(@NotNull Location location, int distance) {
         double maxHeightDiff = 0;
         double maxHeightDiff2 = 0;
         double diff = Math.abs(getHeight(location.clone().add(distance, 0, -distance)) - getHeight(location.clone()

src/main/java/net/onelitefeather/bettergopaint/utils/Sphere.java

@@ -21,6 +21,7 @@
 import org.bukkit.Axis;
 import org.bukkit.Location;
 import org.bukkit.block.Block;
+import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.ArrayList;
@@ -38,7 +39,7 @@ public class Sphere {
      * @param air         Whether air blocks should be included.
      * @return A stream of blocks within the specified radius.
      */
-    public static Stream<Block> getBlocksInRadius(Location middlePoint, int radius, @Nullable Axis axis, boolean air) {
+    public static Stream<Block> getBlocksInRadius(@NotNull Location middlePoint, int radius, @Nullable Axis axis, boolean air) {
         List<Block> blocks = new ArrayList<>();
         Location loc1 = middlePoint.clone().add(-radius / 2d, -radius / 2d, -radius / 2d).getBlock().getLocation();
         Location loc2 = middlePoint.clone().add(radius / 2d, radius / 2d, radius / 2d).getBlock().getLocation();
@@ -91,7 +92,7 @@ public static Stream<Block> getBlocksInRadius(Location middlePoint, int radius,
         return blocks.stream().filter(block -> !block.isEmpty());
     }
 
-    private static boolean passesDefaultChecks(Location location, Location middlePoint, int radius) {
+    private static boolean passesDefaultChecks(@NotNull Location location, @NotNull Location middlePoint, int radius) {
         return location.distance(middlePoint) < radius / 2d;
     }
 

src/main/java/net/onelitefeather/bettergopaint/utils/Surface.java

@@ -22,6 +22,7 @@
 import org.bukkit.Location;
 import org.bukkit.block.Block;
 import org.bukkit.block.BlockFace;
+import org.jetbrains.annotations.NotNull;
 
 public class Surface {
 
@@ -31,7 +32,7 @@ public class Surface {
      * @param block the block to check
      * @return true if the block is on the surface, false otherwise
      */
-    public static boolean isDirectlyOnSurface(Block block) {
+    public static boolean isDirectlyOnSurface(@NotNull Block block) {
         return block.isSolid() && !block.getRelative(BlockFace.UP).isSolid();
     }
 
@@ -42,7 +43,7 @@ public static boolean isDirectlyOnSurface(Block block) {
      * @param playerLoc the player's location
      * @return true if the block is on the surface from the player's location, false otherwise
      */
-    public static boolean isRelativelyOnSurface(Block block, Location playerLoc) {
+    public static boolean isRelativelyOnSurface(@NotNull Block block, @NotNull Location playerLoc) {
         Location location = block.getLocation();
 
         playerLoc.add(0, 1.5, 0);
@@ -81,12 +82,12 @@ public static boolean isRelativelyOnSurface(Block block, Location playerLoc) {
     /**
      * Checks if a given block is on the surface based on the specified surface mode and location.
      *
-     * @param block        the block to check
-     * @param surfaceMode  the surface mode to use for the check
-     * @param location     the location to use for the check
+     * @param block       the block to check
+     * @param surfaceMode the surface mode to use for the check
+     * @param location    the location to use for the check
      * @return true if the block is on the surface based on the surface mode and location, false otherwise
      */
-    public static boolean isOnSurface(Block block, SurfaceMode surfaceMode, Location location) {
+    public static boolean isOnSurface(@NotNull Block block, @NotNull SurfaceMode surfaceMode, @NotNull Location location) {
         return switch (surfaceMode) {
             case RELATIVE -> isRelativelyOnSurface(block, location);
             case DIRECT -> isDirectlyOnSurface(block);