Re: Better Javadoc/Impl for org.apache.logging.log4j.core.Layout.getContentType()

2024-04-12 Thread Matt Sicker
We’ve got nullability annotations now, so we should be defining them. For use cases like this, I’d examine how existing things work and then document the nullability as such. For new code, I prefer things to be non-null except for cases where it makes sense to use null rather than, say, passing

Better Javadoc/Impl for org.apache.logging.log4j.core.Layout.getContentType()

2024-04-12 Thread Gary D. Gregory
Hi All, Our current Javadoc says: /** * Returns the content type output by this layout. The base class returns "text/plain". * * @return the content type. */ String getContentType(); I'd like to clarify if returning null should be legal/allowed for implementation