The Javadoc for the Logger.isEnabled methods looks a bit odd. I just fixed
a few obvious errors, but some questions:

- I guess that the note about Level.OFF applies to both of them?

- Is the marker optional?

- I guess that the marker param should not be formulated like it is?

/**
 * Checks whether this Logger is enabled for the given Level.
 * <p>
 * Note that passing in {@link Level#OFF OFF} always returns {@code true}.
 * </p>
 *
 * @param level the Level to check
 * @return boolean - {@code true} if this Logger is enabled for level,
{@code false} otherwise.
 */
boolean isEnabled(Level level);

/**
 * Checks whether this Logger is enabled for the given Level and an
optional Marker.
 *
 * @param level The Level to check
 * @param marker The marker data specific to this log statement.
 * @return boolean - {@code true} if this Logger is enabled for level
and marker, {@code false} otherwise.
 */
boolean isEnabled(Level level, Marker marker);


-- 
[image: MagineTV]

*Mikael Ståldal*
Senior software developer

*Magine TV*
[email protected]
Grev Turegatan 3  | 114 46 Stockholm, Sweden  |   www.magine.com

Privileged and/or Confidential Information may be contained in this
message. If you are not the addressee indicated in this message
(or responsible for delivery of the message to such a person), you may not
copy or deliver this message to anyone. In such case,
you should destroy this message and kindly notify the sender by reply
email.

Reply via email to