If expand_path/1 already contains LogRoot in its RelPath, do not
add it again.

Before, the old file was still being used until the next
`check_interval`, where the file was recreated and reopened.

Previously, if you configured a trace to use an
extra sink, you'd get a noproc error. Addresses
issue #296 on github.

255 is a valid ISO-8859-1 character. This was fixed
on 2.x by #292. Forward port the fix into 3.x.

* Add formatting options to align text differently if desired
* Fix some typos in comments and docs
* Optionally print all metadata
* Fix R15 compatibility

ex:[{pid, ["123456:", pid], ["Unknown Pid"], {right, 10}}] -> if pid is in the metadata print "   123456:   <?.?.?>", otherwise print "nknown Pid"
placeholders: date, time, message, sev, metadata and new placeholder blank(blank | {blank, Fill} ignore width param

Metadata can be printed with standard or custom internal seperators ad field
seperators. The defaults are "=" and " ":

  "key1=value1 key2=value2"

Based on code review comments put this function into util.
Added a couple simple unit tests for sanity checking purposes.

Removed comment

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.

Fix stop_trace_int so it can remove an event handler

Previously bad configurations would be
silently ignored. Now they will throw
an error tuple.