<div dir="ltr">Yes, there's no such precondition. Thanks for having a look, I updated the javadoc as you suggested, and linked it to the old existing ticket. It's now ready for review.<br><div><br></div><div>I would appreciate if you could have a look again.</div><div><br></div><div>Cheers,</div><div>Valeh</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 14, 2023 at 4:22 PM Archie Cobbs <<a href="mailto:archie.cobbs@gmail.com">archie.cobbs@gmail.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 dir="ltr"><div dir="ltr">On Thu, Dec 14, 2023 at 4:56 AM Viktor Klang <<a href="mailto:viktor.klang@oracle.com" target="_blank">viktor.klang@oracle.com</a>> wrote:</div><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"><div>




<div dir="ltr">
<div><span style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif;font-size:12pt;color:rgb(0,0,0)">I presume that the precondition to have the original collection be pre-ordered according
 to the supplied Comparator can be verified by checking before adding each element in the collection to the PQ that it compareTo equal-or-greater to the previous one?<br></span></div></div></div></blockquote><div><br></div><div>Hmm...  something is not adding up.</div><div><br></div><div>Either (a) there is a precondition that the collection already be sorted or (b) there's no such precondition.<br></div><div><br></div><div>In case (a):</div><div><ul><li>Why isn't the new constructor invoking initElementsFromCollection() instead of initFromCollection()?</li><li>The precondition is not very obvious from the Javadoc description, and moreover what happens when the precondition is not met is not documented at all.<br></li></ul></div><div>In case (b):</div><div><ul><li>The Javadoc is misleading because it (ambiguously) implies there is a precondition with the wording "collection that orders its elements according to the specified comparator" (the referent of "that" is ambiguous - does it refer to the collection or the PriorityQueue?)<br></li></ul></div></div><div class="gmail_quote">From the PR description it seems clear that there is no such precondition. So maybe the Javadoc should say this instead:</div><div class="gmail_quote"><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="gmail_quote">Creates a {@code PriorityQueue} containing the elements in the specified collection. The {@code PriorityQueue} will order its elements according to the specified comparator.</div></blockquote><div class="gmail_quote"><br></div><div class="gmail_quote">-Archie<br clear="all"></div><br><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature">Archie L. Cobbs<br></div></div>
</blockquote></div>