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