LGTM. Could we just get rid of the argumentless one and use a default argument?
Merge the two constructors into one.
Sure, done.
LGTM