The fix looks fine. Thanks.
On 12.08.15 19:51, Stuart Marks wrote:
Hi all,
I still need a second reviewer for this change. Thanks.
s'marks
On 8/3/15 1:49 PM, Phil Race wrote:
CCC is approved. So I will give this a "+1" .. you just need one
other reviewer (if
you did not get that already) and you can push.
-phil.
On 07/27/2015 07:10 PM, Stuart Marks wrote:
Hi all,
Please review this following code and API change:
Bug:
https://bugs.openjdk.java.net/browse/JDK-8068749
Webrev:
http://cr.openjdk.java.net/~smarks/reviews/8068749/webrev.0/
The change is to be pushed into the jdk9/client forest.
The background is that this is a "preparation for Jigsaw" bug. The
javax.imageio.spi.ServiceRegistry class was introduced long ago in
order to
load Image I/O service providers; in fact it was also fully general
service
loading mechanism, capable of loading providers for arbitrary
service types.
This latter function has been superseded by java.util.ServiceLoader
(introduced in JDK 6), although iio.SR can still be used for general
purpose
service provider loading.
In JDK 9, Jigsaw modularization will require additional work, and
use of new,
modular APIs, to implement general purpose service loading. Loading of
specific APIs -- which is the primary use case for iio.SR -- is
considerably
simpler. Given that general purpose service loading is now handled
by j.u.SL,
we wish to remove general purpose support from iio.SR and restrict
its use to
loading only Image I/O related service providers.
This change adds explicit checks in iio.SR to restrict the service
types to
the known set of Image I/O interfaces, and it will throw IAE if
asked to load
a provider for any other service type. Use of iio.SR to load Image I/O
providers will continue to work unchanged.
This is clearly an incompatible change. I've done some internet
searching, and
use of iio.SR to load Image I/O providers is quite common. I was
able to find
one use of iio.SR to load a non Image I/O provider, but the project
where this
occurred migrated to j.u.SL several years ago. Based on this, we
believe the
impact of this change to be small.
s'marks
--
Best regards, Sergey.