Shouldn't you test for empty too?