At the moment, AbstractSourceLocator->locateIdentifiersByType() uses a dodgy "hack" that means the identifier name (the string *) is never actually used, but it seems quite abusive. Find a way around this which means we are just searching by IdentifierType rather than having to provide only an Identifier