A few thoughts and questions on JsonProvider.doLoadProvider(): - "tccl" can be null (in case of system classloader) but thats never really checked - special handling org.apache.geronimo.osgi.locator.ProviderLocator really needed here?
JsonProvider.provider(): - doPrivileged/SecurityManager check really needed here? - method seems thread safe but we do not cache the returned provider instance. Maybe we can to this in a thread local variable? Thanks Hendrik -- Hendrik Saly (salyh, hendrikdev22) @hendrikdev22 PGP: 0x22D7F6EC
