From 102ae5cbd8a9ee80934761d44b9015280ddc60d9 Mon Sep 17 00:00:00 2001 From: TheosRee Date: Thu, 27 Jun 2024 15:35:55 +0200 Subject: [PATCH] remove code comments, see https://github.com/uber/NullAway/issues/801 --- .../quest/event/PrimaryServerThreadComposedEvent.java | 6 +++--- .../betonquest/quest/event/PrimaryServerThreadEvent.java | 4 ++-- .../quest/event/PrimaryServerThreadStaticEvent.java | 4 ++-- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadComposedEvent.java b/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadComposedEvent.java index b01e6d997a..e812334b72 100644 --- a/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadComposedEvent.java +++ b/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadComposedEvent.java @@ -50,16 +50,16 @@ public PrimaryServerThreadComposedEvent(final ComposedEvent synced, final Primar public void execute(@Nullable final Profile profile) throws QuestRuntimeException { call(() -> { synced.execute(profile); - return null; // NullAway false positive: Void -> null + return null; }); } - @SuppressWarnings("NullAway") + @SuppressWarnings("NullAway") // FalsePositive, see https://github.com/uber/NullAway/issues/801 @Override public void execute() throws QuestRuntimeException { call(() -> { synced.execute(); - return null; // NullAway false positive: Void -> null + return null; }); } } diff --git a/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadEvent.java b/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadEvent.java index 9756e9262a..71147b8764 100644 --- a/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadEvent.java +++ b/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadEvent.java @@ -44,12 +44,12 @@ public PrimaryServerThreadEvent(final Event synced, final PrimaryServerThreadDat super(synced, data); } - @SuppressWarnings("NullAway") + @SuppressWarnings("NullAway") // FalsePositive, see https://github.com/uber/NullAway/issues/801 @Override public void execute(final Profile profile) throws QuestRuntimeException { call(() -> { synced.execute(profile); - return null; // NullAway false positive: Void -> null + return null; }); } } diff --git a/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadStaticEvent.java b/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadStaticEvent.java index 426e93afa8..65f3dba87a 100644 --- a/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadStaticEvent.java +++ b/src/main/java/org/betonquest/betonquest/quest/event/PrimaryServerThreadStaticEvent.java @@ -25,12 +25,12 @@ public PrimaryServerThreadStaticEvent(final StaticEvent synced, final PrimarySer super(synced, data); } - @SuppressWarnings("NullAway") + @SuppressWarnings("NullAway") // FalsePositive, see https://github.com/uber/NullAway/issues/801 @Override public void execute() throws QuestRuntimeException { call(() -> { synced.execute(); - return null; // NullAway false positive: Void -> null + return null; }); } }