<div dir="ltr"><div dir="ltr">On Thu, Mar 27, 2025 at 3:03 PM Jaikiran Pai <<a href="mailto:jai.forums2013@gmail.com" target="_blank">jai.forums2013@gmail.com</a>> wrote:</div><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u>

  
    
  
  <div>
    <p>Hello Eirik,</p>
    <p>The code in URLJarFile which does that check for "~" seems to
      have been introduced for
      <a href="https://bugs.openjdk.org/browse/JDK-4447921" target="_blank">https://bugs.openjdk.org/browse/JDK-4447921</a>. It doesn't tell why
      the "~" was necessary.</p>
    <p>In any case, I don't think any kind of clean up or introducing
      tests for "~" is worth it, given it's not clear what that usage
      signifies.</p></div></blockquote><div><br></div><div>Hello Jaikiran,</div><div><br></div><div>My intention is not to fix or improve the handling of "~" per se. It's just a discrepancy which makes it harder to consolidate the logic.</div><div><br></div><div>I should have been clear about what I'm actually trying to improve:</div><div><br></div><div>URLJarFile::isFileURL</div><div>JarFileFactory::urlFor</div><div>unix/s/n/w/p/file/Handler::openConnection</div><div>windows/s/n/w/p/file/Handler::openConnection</div><div><br></div><div>The above four methods all have copies of the "is local file" check. Other than JarFileFactory::urlFor not considering "~", the logic is identical or simply negated. We should be able to the primitive operations and reuse them as utility methods instead.</div><div><br></div><div>The primitive operations here are "isLocalFileURL" and "isNonLocalFileURL".</div><div><br></div><div>To illustrate this possible cleanup, here is a sketch where these two primitives are added to FileURLConnection, then reused from five call sites: </div><div><br></div><div><a href="https://github.com/openjdk/jdk/compare/master...eirbjo:file-url-local-file-cleanup">https://github.com/openjdk/jdk/compare/master...eirbjo:file-url-local-file-cleanup</a></div><div><br></div><div>Thanks,</div><div>Eirik: </div></div></div>
</div>