search

FasterXML / jackson-annotations

Core annotations (annotations that only depend on jackson-core) for Jackson data processor
https://github.com/FasterXML/jackson
Apache License 2.0
1.03k stars 330 forks source link

JsonInclude.Include.NON_EMPTY awkward docs #72

Closed jayanderson closed 8 years ago

jayanderson commented 9 years ago

The current paragraph:

Value that indicates that only properties that have values that values that are null or what is considered empty are not to be included.

I think what's meant is:

Value that indicates that properties with values that are null or are considered empty will not be included.

Or

Properties with null or empty values will not be included when using this value.

Or something similar. This is a very minor issue I just had a hard time parsing the paragraph at first.

cowtowncoder commented 8 years ago

Thanks! Fixed for 2.7 as suggested. Also improved javadoc further along similar lines.