Unnecessary change. Using strlen() abstracts away more of the implementation details of C strings. With the intention to go away from C style, testing the first char for NUL character is a step in the wrong direction. Speed is not a primary concern here either.