> On Sep 26, 2015, at 7:33 AM, Alan Bateman <[email protected]> wrote:
> This looks good. I think I would lean more to deprecating getPackage as just 
> too fragile and leads to difficult to diagnose bugs.

Are you also suggesting to deprecate ClassLoader::getPackages as well?
 
> BTW: I assume the proxy test is a separate change-set and not mean to be in 
> this patch.

That test should be pushed as a separate changeset.  That test reveals a NPE 
thrown by definePackage and helps catching problem.

Mandy

Reply via email to