<html><body><div style="font-family: arial, helvetica, sans-serif; font-size: 12pt; color: #000000"><div>Looks good.<br></div><div><br data-mce-bogus="1"></div><div>Rémi<br data-mce-bogus="1"></div><div><br></div><hr id="zwchr" data-marker="__DIVIDER__"><div data-marker="__HEADERS__"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><b>De: </b>"Brian Burkhalter" <brian.burkhalter@oracle.com><br><b>À: </b>"Remi Forax" <forax@univ-mlv.fr><br><b>Cc: </b>"Alan Bateman" <Alan.Bateman@oracle.com>, "nio-dev" <nio-dev@openjdk.java.net><br><b>Envoyé: </b>Jeudi 8 Février 2018 20:44:36<br><b>Objet: </b>Re: RFR 8191416: (bf) DirectByteBuffer extends MappedByteBuffer, confuses instanceof tests<br></blockquote></div><div data-marker="__QUOTED_TEXT__"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;">On Feb 6, 2018, at 12:33 PM, Remi Forax <<a href="mailto:forax@univ-mlv.fr" target="_blank">forax@univ-mlv.fr</a>> wrote:<br><div><br class="Apple-interchange-newline"><blockquote><blockquote style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Minor comment is that the other methods uses "mapping" in the name so<br>maybe hasFileMapping might be better.<br></blockquote><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;">or remove the method hasMappedFile and inline the body in all the calling methods given it's now a simple == test.<span class="Apple-converted-space"> </span></span></blockquote></div><br><div>Updated patch with hasMappedFile() inlined:</div><br><div><a href="http://cr.openjdk.java.net/~bpb/8191416/webrev.01/" target="_blank">http://cr.openjdk.java.net/~bpb/8191416/webrev.01/</a><br data-mce-bogus="1"></div><br><div>Thanks,</div><br><div>Brian</div><br></blockquote></div></div></body></html>