Misleading resolve error with multiple imports

[brson]

When there is a single import statement that imports multiple idents, and one of them is not found, then the resulting error points to the use site, not the import site.

import http_parser::{http_parser, http_parser_settings, http_parser_execute}; // http_parser_execute is not in the http_parser module

....

http_parser_execute(addr_of(self.parser));

./parser.rs:68:12: 68:31 error: unresolved name: http_parser_execute
./parser.rs:68             http_parser_execute(addr_of(self.parser),

[burg]

This seems to be fixed, but now the error doesn't specify which imported ident is missing, and there may be duplicates for the same missing symbol. I suppose these should be squashed.

/Users/burg/repos/servo/src/rust-http-client/connection.rs:2:7: 2:59 error: failed to resolve import
/Users/burg/repos/servo/src/rust-http-client/connection.rs:2 import std::net::tcp::{tcp_err_data, tcp_connect_err_data};
                                                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/burg/repos/servo/src/rust-http-client/connection.rs:2:7: 2:59 error: failed to resolve import
/Users/burg/repos/servo/src/rust-http-client/connection.rs:2 import std::net::tcp::{tcp_err_data, tcp_connect_err_data};
                                                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

[pnkfelix]

This seems fixed to me. (If you encounter counter-evidence, please reopen, but also throw in the counter-example for us to use for regression test.)