Dry up logging in MailHandler class
The current approach to logging in MailHandler is a bit funny. Instead of checking "
if logger" every time, just make sure
logger returns non-
Also, extra-checking with "
if logger && logger.info" doesn't make much sense. It prints extra blank lines when logger is set and accepts info level, but has no point when level setting is different. It's perfectly safe to just call "