[PATCH] QuadBuckets `toArray` can infinitely recurse

[taylor.smock]

This behavior is somewhat dependent upon the JVM implementation of ArrayList<>(Collection<T> collection).

However, if the constructor uses toArray to get the initial array for the ArrayList, we have something that looks like this:
toArray -> toList -> toArray -> toList -> toArray -> ...

[taylor.smock]

Use this.stream().toArray() instead of this.toList().toArray() for the toArray() function, and document why

[Don-vip]

In 17559/josm:

fix #20587 - fix potential infinite loop in QuadBuckets toArray (patch by taylor.smock)