diff options
| author | Reinier Zwitserloot <reinier@zwitserloot.com> | 2011-10-25 15:01:28 +0200 |
|---|---|---|
| committer | Reinier Zwitserloot <reinier@zwitserloot.com> | 2011-10-25 15:01:28 +0200 |
| commit | 782daa49979c35cd93139b3cfbc98eafe346f790 (patch) | |
| tree | a8f81b23c872fd76164b2d83f6a0448dca529745 /src/utils | |
| parent | c402dd86379e532895f73ee209c432f84bb5f421 (diff) | |
| download | lombok-782daa49979c35cd93139b3cfbc98eafe346f790.tar.gz lombok-782daa49979c35cd93139b3cfbc98eafe346f790.tar.bz2 lombok-782daa49979c35cd93139b3cfbc98eafe346f790.zip | |
Again made @NotNull have no special meaning.
See issues 43, 271, and 287.
Diffstat (limited to 'src/utils')
| -rw-r--r-- | src/utils/lombok/core/TransformationsUtil.java | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/utils/lombok/core/TransformationsUtil.java b/src/utils/lombok/core/TransformationsUtil.java index 3fbfef58..25c3215a 100644 --- a/src/utils/lombok/core/TransformationsUtil.java +++ b/src/utils/lombok/core/TransformationsUtil.java @@ -72,7 +72,11 @@ public class TransformationsUtil { public static final Pattern PRIMITIVE_TYPE_NAME_PATTERN = Pattern.compile( "^(boolean|byte|short|int|long|float|double|char)$"); - public static final Pattern NON_NULL_PATTERN = Pattern.compile("^(?:notnull|nonnull)$", Pattern.CASE_INSENSITIVE); + /* NB: 'notnull' is not part of the pattern because there are lots of @NotNull annotations out there that are crappily named and actually mean + something else, such as 'this field must not be null _when saved to the db_ but its perfectly okay to start out as such, and a no-args + constructor and the implied starts-out-as-null state that goes with it is in fact mandatory' which happens with javax.validation.constraints.NotNull. + Various problems with spring have also been reported. See issue #287, issue #271, and issue #43. */ + public static final Pattern NON_NULL_PATTERN = Pattern.compile("^(?:nonnull)$", Pattern.CASE_INSENSITIVE); public static final Pattern NULLABLE_PATTERN = Pattern.compile("^(?:nullable|checkfornull)$", Pattern.CASE_INSENSITIVE); /** |