Tidy up Logger in preparation for optimizations.
Highlights:
- Clean up documentation. Moved internal docs from variable use site
to declaration site, especially for special cases around null.
- Move default variable values to declaration
- Move methods requiring LogManager synchronization to LogManager.
This includes setLevelImpl(), forceChildsToInherit(),
internalSetParent() and getLoggerWithRes().
- flip "if (null == foo)" to "if (foo == null)" for readability
- Rename updateResourceBundle to initResourceBundle. Made method
non-static.
- Rename initHandler to initHandlers; changed the method to
short circuit on negative 'handlerInited' cases.
- Flipped "internalIsLoggable" methods to short-circuit.
- Expose variables used by LogManager as package-private. For example,
the childs variable is only used by LogManager, but previously it
went through the Logger class each time it was used.
2 files changed