Thanks - looks good.

---

+    private final static List<String> cals =
+        List.of("gregorian", "japanese", "julian");

If you inline this into main, your beautiful stream pipeline will be even
more beautiful!

---

+import static java.util.Calendar.Builder;

My colleagues would frown upon static importing a builder - just use
"Calendar.Builder" !



On Wed, May 31, 2017 at 12:41 PM, Naoto Sato <naoto.s...@oracle.com> wrote:

> Hi Martin,
>
> These are the issues you filed with proposed fixes. I supplemented the
> tests to them. Can you please review these changes:
>
> https://bugs.openjdk.java.net/browse/JDK-8176160
> http://cr.openjdk.java.net/~naoto/8176160/webrev.00/
>
> https://bugs.openjdk.java.net/browse/JDK-8176847
> http://cr.openjdk.java.net/~naoto/8176847/webrev.00/
>
> https://bugs.openjdk.java.net/browse/JDK-8176853
> http://cr.openjdk.java.net/~naoto/8176853/webrev.00/
>
> Naoto
>
>

Reply via email to