Minor optimisations in select.cc's match_ids function

Minor optimisations in select.cc's match_ids function

  - Pass a string argument by reference

  - Avoid multiple calls to IdString::str and IdString::c_str

  - Avoid combining checks for size > 0 and first char (C strings are
    null terminated, so foo[0] != '\0' implies that foo has positive
    length)