`empty()` explain better the meaning.
`empty()` is, for some container (as list), faster than `size() > 0`. (it is not the case for vector).
That line was flagged by cppcheck.

I have the habit to use explicitly `true/false` but `!names.empty()` is good too.
I will change that this week-end.