<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;">Don’t want to push or force you, just curious: how is it going? :)</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">How much time it usually takes to review such patches?</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">P.S. I have got commit access few days ago, just fyi</div> <div id="bloop_sign_1424107796065107968" 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 19 Jan 2015 at 18:29:51, 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;"><div></div><div>
<title></title>
Sorry, yes busy and vacation. Your patch looks good to me. I have a
pending proposal which pretty much
<div>describes what is implemented (with due credit :). Let’s wait
for outcome of this proposal.</div>
<div><br></div>
<div>- Thanks, Fariborz</div>
<div><br>
<div>
<div>On Jan 17, 2015, at 1:44 PM, AlexDenisov <<a href="mailto:1101.debian@gmail.com">1101.debian@gmail.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite">
<div 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; 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; margin: 0px;">I
guess you’re busy with the next LLVM/Clang release, but just want
to know how is it going with this patch.</div>
<div id="bloop_customfont" style="font-family: Helvetica, Arial; font-size: 13px; margin: 0px;">Let
me know if I can help somehow.</div>
<div id="bloop_sign_1421530465535212032" 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="">On 10 Jan 2015 at 11:12:39, AlexDenisov (<a href="mailto:1101.debian@gmail.com">1101.debian@gmail.com</a>)
wrote:</p>
<blockquote type="cite" class="clean_bq">
<div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">
<div>
<div id="bloop_customfont" style="font-family: Helvetica, Arial; font-size: 13px; margin: 0px;">
<div id="bloop_customfont" style="margin: 0px;"><span>Here is new
version, without pointers.</span></div>
<div id="bloop_customfont" style="margin: 0px;">
<span><br></span></div>
</div>
<div id="bloop_sign_1420884482362669824" class="bloop_sign">
<div style="font-family: helvetica, arial; font-size: 13px;">
<span>-- <br>
AlexDenisov</span></div>
<div style="font-family: helvetica, arial; font-size: 13px;">
<span>Software Engineer, <a href="https://github.com/AlexDenisov">https://github.com/AlexDenisov</a></span></div>
</div>
<span><br></span>
<p style=""><span>On 9 Jan 2015 at 20:14:47, jahanian (<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>)
wrote:</span></p>
<blockquote type="cite" class="clean_bq">
<div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">
<div class=""><span><span><br class=""></span></span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;"><span>We may
not support boxing of any pointer types after
all </span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;"><span>(as the
underlying API does not manage the collected object).</span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;"><span>So, final
patch may need to address that. Please provide more
tests </span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;"><span>for
NSEdgeInsets and specifically, see that diagnostics come out
when </span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">
<span>deployment target does not support it.</span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">
<span>Otherwise, patch looks good. Please hang on to the patch
until we have gone through</span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;"><span>the
language review process.</span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">
<span><br class=""></span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;"><span>- Thanks,
Fairborz</span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">
<span><br class=""></span></div>
<div class="" style="margin: 0px; font-size: 11px; font-family: Menlo;">
<span><br class=""></span></div>
<div>
<blockquote type="cite" class="">
<div class=""><span>On Jan 6, 2015, at 10:23 AM, AlexDenisov
<<a href="mailto:1101.debian@gmail.com" class="">1101.debian@gmail.com</a>> wrote:</span></div>
<span><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;">
<span>Thank you for response. </span></div>
<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;">
<span>I already sent another patch, it could be found
here: </span></div>
<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;">
<span><a href="http://article.gmane.org/gmane.comp.compilers.clang.scm/114023" class="">http://article.gmane.org/gmane.comp.compilers.clang.scm/114023</a></span></div>
<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;">
<span><br class=""></span></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 class="" style="font-family: helvetica, arial; font-size: 13px;">
<span>-- <br class="">
AlexDenisov</span></div>
<div class="" style="font-family: helvetica, arial; font-size: 13px;"><span>Software
Engineer, <a href="https://github.com/AlexDenisov" class="">https://github.com/AlexDenisov</a></span></div>
</div>
<span><br 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;">
</span>
<p 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;">
<span>On 5 Jan 2015 at 19:39:16, jahanian (<a href="mailto:fjahanian@apple.com" class="">fjahanian@apple.com</a>)
wrote:</span></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><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>
</div>
</blockquote>
</div>
</div>
</blockquote>
</div>
</blockquote>
</div>
<br></div>
</div></div></span></blockquote></body></html>