Functionally there is no change to the code by this change addresses
the "buffer not null terminated" warning in the logger constructor.
As well as removing the cause of the warning (a "strncpy" where
the length copied is equal to the destination buffer length), it
it removed two assert() calls: one was replaced by the throwing of
am exception, and the second was redundant.