> On Jan 11, 2017, at 5:47 PM, Paul Sandoz <paul.san...@oracle.com> wrote: > > JmodTask > — > > 839 > 840 hashesBuilder.computeHashes(modules).entrySet().forEach(e -> { > > You can use the Map.forEach method accepting BiFunction of key and value. >
Good suggestion. Fixed. > > Make static? > It could but the Hasher class needs to be made static too and pass in other instance variables from JmodTask. There are other inner classes in JmodTask that accesses JmodTask instance variables. I prefer to leave this for a future clean up to make the implementation consistent, if this is okay with you? Mandy