RFR: 8215995: Add specialized toArray methods to immutable collections
Stuart Marks
stuart.marks at oracle.com
Tue Jan 8 02:44:41 UTC 2019
> This is one of Stuart's designs
Trying to blame it all on me, aren't you? :-)
Overall the changes look good.
I can't really speak to how to improve the loops in certain circumstances (the
"tricky circular array traversal code from ArrayDeque" as Martin puts it). But I
observe that moving the randomization to iteration has started to increase the
complexity of the code. I understand why we did this, but it's starting to look
like we might want to refactor the various loops into a single internal
"iterator" to avoid duplicating it all over the place. Perhaps for a future
enhancement.
I observe that the serialized forms aren't randomized. When the storage order
was randomized, the serial form was also randomized. But now that randomization
is done during iteration, the serialized forms are fixed. Hmmm. Not sure what I
think about that.
s'marks
More information about the core-libs-dev
mailing list