shouldn't empty string be default?