During code review we decided that using Sink ++ '_lager_event'
would provide better isolation for gen_event names than
Sink ++ '_event' as we had been using up to this point.
For example a sink called 'audit' is now 'audit_lager_event'
than 'audit_event' which might clash with a user's predefined
own module or process name.
The example in "Internal log rotation" includes the tuple
{name, "error.log"}
This apparently should be
{file, "error.log"}
as seen in other config examples in README.md.
The current set of instructions mention 'crash_log' being left undefined
as a variable to disable it, but when that happens, lager starts it at a
default location.
This commit makes it obvious that the value must explicitly set to
'undefined' in order to disable crash_log.