<div dir="ltr">As a minor consumer of the API, both changes seem reasonable to me though I have yet to need to use either set of classes.<div><br></div><div>Given "jdktypes" isn't a particularly descriptive package name, drawing the parallel for PackageDesc/ModuleDesc to java.lang.constant makes sense.<div><br></div><div>--Dan</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 1, 2023 at 4:25 AM Adam Sotona <<a href="mailto:adam.sotona@oracle.com">adam.sotona@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="msg4186501797993477793">





<div lang="en-CZ" style="overflow-wrap: break-word;">
<div class="m_4186501797993477793WordSection1">
<p class="MsoNormal"><span lang="EN-US">Hi,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Based on PR feedback I would like to propose two possible API changes:<u></u><u></u></span></p>
<ol style="margin-top:0cm" start="1" type="1">
<li class="m_4186501797993477793MsoListParagraph" style="margin-left:0cm"><span lang="EN-US">Rename
</span><span lang="EN-US" style="font-family:Consolas">jdk.internal.classfile.jdktypes</span><span lang="EN-US"> package to
</span><span lang="EN-US" style="font-family:Consolas">jdk.internal.classfile.constant</span><span lang="EN-US"> to reflect the fact that
</span><span lang="EN-US" style="font-family:Consolas">PackageDesc</span><span lang="EN-US"> and
</span><span lang="EN-US" style="font-family:Consolas">ModuleDesc</span><span lang="EN-US"> are complementing
</span><span lang="EN-US" style="font-family:Consolas">java.lang.constant</span><span lang="EN-US"> package content.<u></u><u></u></span></li><li class="m_4186501797993477793MsoListParagraph" style="margin-left:0cm"><span lang="EN-US">Move
</span><span lang="EN-US" style="font-family:Consolas">Signature</span><span lang="EN-US">,
</span><span lang="EN-US" style="font-family:Consolas">MethodSignature</span><span lang="EN-US"> and
</span><span lang="EN-US" style="font-family:Consolas">ClassSignature</span><span lang="EN-US"> from
</span><span lang="EN-US" style="font-family:Consolas">jdk.internal.classfile</span><span lang="EN-US"> package to the same package as #1, as the signature models are similar to descriptor models and serve the same purpose.<u></u><u></u></span></li></ol>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Please let me know if you agree or disagree individually with #1 and #2.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Thank you,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Adam<u></u><u></u></span></p>
</div>
</div>

</div></blockquote></div>