Fixed Method invocation 'getX' may produce 'NullPointerException'

The warning occured because `delivery.getBlockPos()` might return null, leading to a `NullPointerException` when calling `pos.getX()`. Although `isBlockTarget()` checks for non-null `blockPos`, the static analyzer doesn't recognize this invariant.

Solved it by adding a null check for `delivery.getBlockPos()` in the `applyTargeted` method. Only proceed with sound playback if `pos` is not null. The null check if `(pos != null)` ensures `pos.getX()` won't be called on a null reference. This satisfies the static analyzer while maintaining the existing logic since `isBlockTarget()` already implies `blockPos` shouldn't be null. The check acts as a safeguard against potential future changes to `TargetedSpellDelivery`.

Author: 0vergrown

src/main/java/dev/overgrown/thaumaturge/spell/impl/victus/VictusEffect.java

@@ -35,7 +35,9 @@ public void applyTargeted(TargetedSpellDelivery delivery) {
             applyVictusEffect(target, getPowerMultiplier(delivery.getModifiers()), delivery.getCaster());
         } else if (delivery.isBlockTarget()) {
             BlockPos pos = delivery.getBlockPos();
-            playSound(delivery.getWorld(), pos.getX() + 0.5, pos.getY() + 0.5, pos.getZ() + 0.5);
+            if (pos != null) { // Null check added here
+                playSound(delivery.getWorld(), pos.getX() + 0.5, pos.getY() + 0.5, pos.getZ() + 0.5);
+            }
         }
     }