<html><head><style>body{font-family:Helvetica,Arial;font-size:13px}</style></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><div><blockquote type="cite" class="clean_bq" style="color: rgb(0, 0, 0); font-family: Helvetica, Arial; font-size: 13px; 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;"><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">We may not support boxing of any pointer types after all </div><div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">(as the underlying API does not manage the collected object).</div><div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">So, final patch may need to address that.</div></div></blockquote></div><p>No problem. To be honest, I don’t see real benefits of having such ability, it was a silly decision to include this functionality (motivated by ‘because I can’).</p><div><blockquote type="cite" class="clean_bq" style="color: rgb(0, 0, 0); font-family: Helvetica, Arial; font-size: 13px; 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;"><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">Please provide more tests </div><div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">for NSEdgeInsets and specifically, see that diagnostics come out when </div><div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">deployment target does not support it.</div></div></blockquote></div><p>What kind of tests should I add? It has tests for Sema and CodeGen, did I forget/missed something?</p><div><div><blockquote type="cite" class="clean_bq" style="color: rgb(0, 0, 0); font-family: Helvetica, Arial; font-size: 13px; 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;"><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">Please hang on to the patch until we have gone through</div><div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">the language review process.</div></div></blockquote></div><p>Got it!</p></div></div> <div id="bloop_sign_1420831867749383936" class="bloop_sign"><div style="font-family:helvetica,arial;font-size:13px">-- <br>AlexDenisov</div><div style="font-family:helvetica,arial;font-size:13px">Software Engineer, <a href="https://github.com/AlexDenisov">https://github.com/AlexDenisov</a></div></div> <br><p style="color:#000;">On 9 Jan 2015 at 20:14:47, jahanian (<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>) wrote:</p> <blockquote type="cite" class="clean_bq"><span><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div></div><div>
<title></title>
<div class=""><br class=""></div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class="">We may not support boxing of any pointer types after
all </div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class="">(as the underlying API does not manage the collected
object).</div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class="">So, final patch may need to address that. Please provide
more tests </div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class="">for NSEdgeInsets and specifically, see that diagnostics
come out when </div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class="">deployment target does not support it.</div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class="">Otherwise, patch looks good. Please hang on to the patch
until we have gone through</div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class="">the language review process.</div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class=""><br class=""></div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class="">- Thanks, Fairborz</div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class=""><br class=""></div>
<div style="margin: 0px; font-size: 11px; font-family: Menlo;" class=""><br class=""></div>
<div>
<blockquote type="cite" class="">
<div class="">On Jan 6, 2015, at 10:23 AM, AlexDenisov <<a href="mailto:1101.debian@gmail.com" class="">1101.debian@gmail.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div id="bloop_customfont" style="font-family: Helvetica, Arial; font-size: 13px; 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; margin: 0px;" class="">Thank you for response. </div>
<div id="bloop_customfont" style="font-family: Helvetica, Arial; font-size: 13px; 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; margin: 0px;" class="">I already sent another patch, it could be found
here: </div>
<div id="bloop_customfont" style="font-family: Helvetica, Arial; font-size: 13px; 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; margin: 0px;" class=""><a href="http://article.gmane.org/gmane.comp.compilers.clang.scm/114023" class="">http://article.gmane.org/gmane.comp.compilers.clang.scm/114023</a></div>
<div id="bloop_customfont" style="font-family: Helvetica, Arial; font-size: 13px; 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; margin: 0px;" class=""><br class=""></div>
<div id="bloop_sign_1420568589090018048" class="bloop_sign" style="font-family: Helvetica, Arial; font-size: 13px; 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;">
<div style="font-family: helvetica, arial; font-size: 13px;" class="">-- <br class="">
AlexDenisov</div>
<div style="font-family: helvetica, arial; font-size: 13px;" class="">Software Engineer, <a href="https://github.com/AlexDenisov" class="">https://github.com/AlexDenisov</a></div>
</div>
<br style="font-family: Helvetica, Arial; font-size: 13px; 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;" class="">
<p style="font-family: Helvetica, Arial; font-size: 13px; 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;" class="">On 5 Jan 2015 at 19:39:16, jahanian (<a href="mailto:fjahanian@apple.com" class="">fjahanian@apple.com</a>)
wrote:</p>
<blockquote type="cite" class="clean_bq" style="font-family: Helvetica, Arial; font-size: 13px; 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;">
<div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">
<div class=""></div>
<div class=""><span class=""><br class=""></span>
<div class="">
<blockquote type="cite" class="">
<div class=""><span class="">On Dec 24, 2014, at 3:29 AM,
AlexDenisov <<a href="mailto:1101.debian@gmail.com" class="">1101.debian@gmail.com</a>> wrote:</span></div>
<span class=""><br class="Apple-interchange-newline"></span>
<div class="">
<div id="bloop_customfont" class="" style="font-family: Helvetica, Arial; font-size: 13px; 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; margin: 0px;">
<div class="">
<blockquote type="cite" class="clean_bq" style="font-family: Helvetica, Arial; font-size: 13px; 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;">
<div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">
<div class=""><span class="">Do the usual lookup to find the method
which implements this syntax. Call <span class="" style="font-family: Menlo; font-size: 11px; color: rgb(79, 129, 135);">Decl</span><span class="" style="font-family: Menlo; font-size: 11px;">::getAvailability on
this method. If it returns</span></span></div>
<div class=""><span class="" style="font-family: Menlo; font-size: 11px;">anything other
than </span><span class="" style="font-family: Menlo; font-size: 11px;">AvailabilityResult::</span><font face="Menlo" class=""><span class="" style="font-size: 11px;">AR_Available </span></font></div>
</div>
</blockquote>
</div>
<p class="">Well, I understand this part, it’s pretty obvious. But
what I don’t understand is:</p>
<div class="">
<div class="">
<blockquote type="cite" class="clean_bq" style="font-family: Helvetica, Arial; font-size: 13px; 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;">
<div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">
<font face="Menlo" class=""><span class="" style="font-size: 11px;">issue an appropriate
diagnostic.</span></font></div>
</blockquote>
</div>
<p class="">What is appropriate diagnostic for <span class="" style="color: rgb(49, 89, 93); font-family: Menlo; font-size: 11px;">AR_NotYetIntroduced?</span></p>
<div class="">Should I introduce this diagnostic? Or just show
warning/error for Deprecated/Unavailable and ignore
NotYetIntroduced AR?</div>
</div>
</div>
</div>
</blockquote>
<div class=""><br class=""></div>
It is sufficient to issue an unavailability diagnostics since
diagnostic points to the method which has the availability info. as
part of its declaration.</div>
<div class=""><br class=""></div>
<div class="">- Fariborz</div>
</div>
</div>
</blockquote>
</div>
</blockquote>
</div>
<br class="">
</div></div></span></blockquote></body></html>