RFR 8225482: Refactoring: Remove Filter<Symbol> from Scope.getSymbols(|ByName) signatures

[Ron Shapiro]

Hi,

Please review this refactoring to remove Filter<Symbol> from
Scope.getSymbols() and Scope.getSymbolsByName().

webrev: http://cr.openjdk.java.net/~ronsh/8225482/webrev.00/
bug: https://bugs.openjdk.java.net/browse/JDK-8225482

There is one failing jtreg test, and I'm trying to understand what it is.
I'm not sure how to debug it, but I'll report back with anything I find.

tools/javac/processing/model/completionfailure/MissingClassFile.java

com.sun.tools.javac.code.ClassFinder$BadClassFile: bad class file:
./lib-classes/pkg/A$I.class
  illegal start of class file

  Please remove or make sure it appears in the correct subdirectory of the
classpath.
        at
jdk.compiler/com.sun.tools.javac.jvm.ClassReader.badClassFile(ClassReader.java:296)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20190607/89c47813/attachment.html>