I would probably test for `m ~= "NONE"` instead of `missile_names[m]`, because I think that makes it clearer why the test is there, but I don't really mind.