JDK 9 RFR of JDK-8173609: Elements.printElements needs to support modules
Remi Forax
forax at univ-mlv.fr
Sun Jan 29 11:21:59 UTC 2017
Hi Joe,
i believe that all non public methods you have added can be declared private,
and printNameableList can use a stream and a collector
private void printNameableList(List<? extends QualifiedNameable> nameables) {
writer.print(nameables.stream().map(QualifiedNameable::getQualifiedName).collect(Collectors.joining(", "));
}
cheers,
Rémi
----- Mail original -----
> De: "joe darcy" <joe.darcy at oracle.com>
> À: compiler-dev at openjdk.java.net
> Envoyé: Samedi 28 Janvier 2017 22:39:26
> Objet: JDK 9 RFR of JDK-8173609: Elements.printElements needs to support modules
> Hello,
>
> Please review the straightforward fix to address:
>
> JDK-8173609: Elements.printElements needs to support modules
> http://cr.openjdk.java.net/~darcy/8173609.0/
>
> Once additional information about modules is available (JDK-8172810:
> ModuleElement should declare and provide appropriate modifiers), the
> printing processor should be updated to expose that information.
>
> Admittedly this changeset would be better with some regression testing.
>
> Thanks,
>
> -Joe
More information about the compiler-dev
mailing list