From e2f800dd631240a4459ed2c095859b9346d58471 Mon Sep 17 00:00:00 2001 From: Luck Date: Wed, 26 Jul 2017 10:10:25 +0100 Subject: [PATCH] Throw an NPE if a null delegate sender is passed to SenderFactory#wrap --- .../luckperms/common/commands/sender/SenderFactory.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/common/src/main/java/me/lucko/luckperms/common/commands/sender/SenderFactory.java b/common/src/main/java/me/lucko/luckperms/common/commands/sender/SenderFactory.java index 29550816..4f17e968 100644 --- a/common/src/main/java/me/lucko/luckperms/common/commands/sender/SenderFactory.java +++ b/common/src/main/java/me/lucko/luckperms/common/commands/sender/SenderFactory.java @@ -27,6 +27,7 @@ package me.lucko.luckperms.common.commands.sender; import lombok.AccessLevel; import lombok.Getter; +import lombok.NonNull; import lombok.RequiredArgsConstructor; import me.lucko.luckperms.api.Tristate; @@ -59,7 +60,7 @@ public abstract class SenderFactory { protected abstract boolean hasPermission(T t, String node); - public final Sender wrap(T t) { - return new AbstractSender<>(plugin, this, t); + public final Sender wrap(@NonNull T sender) { + return new AbstractSender<>(plugin, this, sender); } }