<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
On 19/12/2012 21:35, Sean Mullan wrote:
<blockquote cite="mid:50D23325.1090909@oracle.com" type="cite">For
JEPs 161 and 162, we need to remove or deprecate undesirable
dependencies from the compact1 profile.
<br>
<br>
One of these is the dependency on
com.sun.security.auth.login.ConfigFile from
sun.security.provider.ConfigSpiFile.
<br>
<br>
The proposed fix moves the guts of the ConfigFile implementation
to sun.security.provider.ConfigSpiFile, and changes ConfigFile
such that it is simply a wrapper around ConfigSpiFile.
<br>
<br>
webrev:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~mullan/webrevs/8005117/webrev.00/">http://cr.openjdk.java.net/~mullan/webrevs/8005117/webrev.00/</a>
<br>
</blockquote>
Thanks for doing this, it's good to get this dependency fixed.<br>
<br>
I've looked at the webrev and I can't see anything obviously wrong
(meaning it looks like you've transposed the code correctly). I
think once the tests pass then you are good to go.<br>
<br>
Once thing in passing is that I see this code uses legacy
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
<span class="changed">StreamTokenizer</span>, might be good to get
it moved to java.util.regex some day.<br>
<br>
-Alan<br>
<br>
<br>
</body>
</html>