In the existing code, if the message was an empty string instead, it would still pass the ```!m.endsWith("\n")``` condition and append a newline.   An empty string and a null message are equally useful (useless) and should probably behave the same.   Either way lets avoid the NPE  :)