diff options
| author | Reinier Zwitserloot <reinier@zwitserloot.com> | 2013-09-24 01:18:32 +0200 |
|---|---|---|
| committer | Reinier Zwitserloot <reinier@zwitserloot.com> | 2013-09-24 01:18:32 +0200 |
| commit | 86a635876dd75c4f3a61593491fa2ce53f8444b8 (patch) | |
| tree | 86b3a14390f64fcc61375da89bd62764d73a867f /src/core/lombok/eclipse/handlers/NonNullHandler.java | |
| parent | 7ee868659f4ff3cb286b676d649e8c57e9248d87 (diff) | |
| parent | 72b55dccb18f38b8aefd0ac8e7c2e8bd2dd5c057 (diff) | |
| download | lombok-86a635876dd75c4f3a61593491fa2ce53f8444b8.tar.gz lombok-86a635876dd75c4f3a61593491fa2ce53f8444b8.tar.bz2 lombok-86a635876dd75c4f3a61593491fa2ce53f8444b8.zip | |
Merge branch 'master' into jdk8check
Conflicts:
src/core/lombok/core/Version.java
src/core/lombok/javac/handlers/HandleConstructor.java
test/transform/resource/messages-delombok/EqualsAndHashCodeWithSomeExistingMethods.java.messages
test/transform/resource/messages-ecj/EqualsAndHashCodeWithSomeExistingMethods.java.messages
Diffstat (limited to 'src/core/lombok/eclipse/handlers/NonNullHandler.java')
| -rw-r--r-- | src/core/lombok/eclipse/handlers/NonNullHandler.java | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/core/lombok/eclipse/handlers/NonNullHandler.java b/src/core/lombok/eclipse/handlers/NonNullHandler.java index 5c58069c..59fda801 100644 --- a/src/core/lombok/eclipse/handlers/NonNullHandler.java +++ b/src/core/lombok/eclipse/handlers/NonNullHandler.java @@ -82,6 +82,11 @@ public class NonNullHandler extends EclipseAnnotationHandler<NonNull> { if (isGenerated(declaration)) return; + if (declaration.isAbstract()) { + annotationNode.addWarning("@NonNull is meaningless on a parameter of an abstract method."); + return; + } + // Possibly, if 'declaration instanceof ConstructorDeclaration', fetch declaration.constructorCall, search it for any references to our parameter, // and if they exist, create a new method in the class: 'private static <T> T lombok$nullCheck(T expr, String msg) {if (expr == null) throw NPE; return expr;}' and // wrap all references to it in the super/this to a call to this method. |