This method looks wrong: the arg `notOfType` is a very specific name, and the method body doesn't use it.
What does it add over calling `Collections2.filter(...)` directly?

Also, for future reference in terms of good generics usage it would be better to do: `<T extends Location> Collection<T> filter(Collection<T> locations, Predicate<? super T> predicate)` so you get back the type that you passed in.