<div dir="ltr">Hi,<br><div><br></div><div>At the end of s.n.w.p.j.JarURLConnection::connect, I see this stray, permission related comment:</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font face="monospace">/* we also ask the factory the permission that was required<br> * to get the jarFile, and set it as our permission.<br> */<br>if (useCaches) {<br>    boolean oldUseCaches = jarFileURLConnection.getUseCaches();<br>    jarFileURLConnection = factory.getConnection(jarFile);<br>    jarFileURLConnection.setUseCaches(oldUseCaches);<br>}</font></blockquote><div><br></div><div><div>The "factory" here is JarFileFactory, which recently went through a SM-cleanup PR which removed permission checking.</div><div></div></div><div><br></div><div>The comment seems to have existed since the initial load, but have moved around a bit since then.</div><div><br></div><div>It does not seem to make a lot of sense now, and digging through history I'm also struggling a bit to understand how it made sense even in the initial load.</div><div><br></div><div>What's the best action here? Delete the comment? Replace it with something more appropriate? Wait for pending JEP486 cleanup to take care of it?</div><div><br></div><div>Thanks,</div><div>Eirik.</div></div>