RFR: Here are some URLClassPath patches
Roger.Riggs at Oracle.com
Wed Feb 21 15:48:30 UTC 2018
Generally, I would leave code alone that does not have a good reason for
That goes for commenting conventions (// vs /*) too.
On the style changes in URLClassPath-ArrayDeque, about declarations like:
ArrayList<URL> path = new ArrayList<>();
It had been a recommended style to use the abstract type (List) on the
left hand side
and only use the concrete type where necessary.
But with the addition of 'var' local type inference, perhaps that would
an option here.
(Not trying to start a style war).
On 2/21/2018 1:08 AM, Martin Buchholz wrote:
> At Google I spend a lot of time staring unproductively at classloader code,
> and it's always hard for me to resist the urge to clean it up. So here are
> some patches that should be relatively uncontroversial, and may prepare for
> more radical changes later.
> 8198484: URLClassPath should use an ArrayDeque instead of a Stack
More information about the core-libs-dev