Remove irrelevant or outdated todos [skip ci]

Author: Wide-Cat

src/main/java/meteordevelopment/meteorclient/gui/screens/ContainerInventoryScreen.java

@@ -29,7 +29,7 @@
 import static meteordevelopment.meteorclient.MeteorClient.mc;
 
 /*
- * i couldn't figure out how to add proper outer borders for the GUI without adding custom textures. @TODO
+ * i couldn't figure out how to add proper outer borders for the GUI without adding custom textures.
  */
 public class ContainerInventoryScreen extends Screen {
     private static final Identifier SLOT_TEXTURE = Identifier.ofVanilla("container/slot");

src/main/java/meteordevelopment/meteorclient/gui/widgets/containers/WView.java

@@ -110,7 +110,7 @@ public void onMouseMoved(double mouseX, double mouseY, double lastMouseX, double
 
             //scroll += Math.round(theme.scale(mouseDelta + mouseDelta * ((height / actualHeight) * 0.7627725)));
             //scroll += Math.round(theme.scale(mouseDelta * (1 / (height / actualHeight))));
-            scroll += Math.round(mouseDelta * ((actualHeight - handleHeight() / 2) / height)); // TODO: Someone improve this
+            scroll += Math.round(mouseDelta * ((actualHeight - handleHeight() / 2) / height));
             scroll = MathHelper.clamp(scroll, 0, actualHeight - height);
 
             targetScroll = scroll;

src/main/java/meteordevelopment/meteorclient/mixin/ItemEntityRendererMixin.java

@@ -30,7 +30,6 @@ public abstract class ItemEntityRendererMixin {
 
     @Inject(method = "render(Lnet/minecraft/client/render/entity/state/ItemEntityRenderState;Lnet/minecraft/client/util/math/MatrixStack;Lnet/minecraft/client/render/command/OrderedRenderCommandQueue;Lnet/minecraft/client/render/state/CameraRenderState;)V", at = @At("HEAD"), cancellable = true)
     private void renderStack(ItemEntityRenderState itemEntityRenderState, MatrixStack matrixStack, OrderedRenderCommandQueue orderedRenderCommandQueue, CameraRenderState arg, CallbackInfo ci) {
-        // todo fix to work with entity render batching
         RenderItemEntityEvent event = MeteorClient.EVENT_BUS.post(RenderItemEntityEvent.get(itemEntityRenderState, mc.getRenderTickCounter().getTickProgress(true), matrixStack, null, itemEntityRenderState.light, this.itemModelManager, orderedRenderCommandQueue));
         if (event.isCancelled()) ci.cancel();
     }

src/main/java/meteordevelopment/meteorclient/utils/Utils.java

@@ -246,7 +246,6 @@ public static boolean openContainer(ItemStack itemStack, ItemStack[] contents, b
         return false;
     }
 
-    @SuppressWarnings("deprecation") // Use of NbtCompound#getNbt
     public static void getItemsInContainerItem(ItemStack itemStack, ItemStack[] items) {
         if (itemStack.getItem() == Items.ENDER_CHEST) {
             for (int i = 0; i < EChestMemory.ITEMS.size(); i++) {
@@ -267,8 +266,6 @@ public static void getItemsInContainerItem(ItemStack itemStack, ItemStack[] item
                 if (i >= 0 && i < items.length) items[i] = stacks.get(i);
             }
         }
-        // todo should we remove this? are there still instances where we might get presented container items in this
-        //  format? maybe on servers with weird multiversion setups - if they exist, test this code to ensure it works
         else if (components.contains(DataComponentTypes.BLOCK_ENTITY_DATA)) {
             TypedEntityData<BlockEntityType<?>> blockEntityData = components.get(DataComponentTypes.BLOCK_ENTITY_DATA);
             if (blockEntityData == null) return;
@@ -310,7 +307,6 @@ public static Color getShulkerColor(ItemStack shulkerItem) {
         return WHITE;
     }
 
-    @SuppressWarnings("deprecation") // Use of NbtCompound#getNbt
     public static boolean hasItems(ItemStack itemStack) {
         ContainerComponentAccessor container = ((ContainerComponentAccessor) (Object) itemStack.get(DataComponentTypes.CONTAINER));
         if (container != null && !container.meteor$getStacks().isEmpty()) return true;