
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jul 27, 2012, at 4:27 PM, Jordan Rose wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=us-ascii"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jul 27, 2012, at 16:23 , jahanian <<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>> wrote:</div><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><font class="Apple-style-span" color="#1864c5"><br></font></div><div>I wonder if leaving the "followsCreateRule" after the CFAuditedTransferAttr check would be better; relying on the naming convention for un-audited code may be a bad idea even</div></div></blockquote><div><br></div>If I leave it after, then CFAuditedTransferAttr check would not get me to check for the naming convention. Note that since in the non-diagnose case they both return</div><div>ACC_invalid, this will not change the behavior.</div></div></blockquote><div><br></div><div>That's intentional. In our real headers, we have CFAuditedTransferAttrs on all parts of CoreFoundation that are guaranteed to correctly follow the naming conventions. CFDictionary.h, for example, is now bracketed in <span style="font-family: Menlo; font-size: 11px; color: rgb(120, 73, 42); ">_Pragma(</span><span style="color: rgb(209, 47, 27); font-family: Menlo; font-size: 11px; ">"clang arc_cf_code_audited begin"</span><span style="font-family: Menlo; font-size: 11px; color: rgb(120, 73, 42); ">)</span> / <span style="font-family: Menlo; font-size: 11px; color: rgb(120, 73, 42); ">_Pragma(</span><span style="color: rgb(209, 47, 27); font-family: Menlo; font-size: 11px; ">"clang arc_cf_code_audited end"</span><span style="font-family: Menlo; font-size: 11px; color: rgb(120, 73, 42); ">)</span>, which implicitly adds the attribute to everything in between. We should probably have this in our fixit test cases.</div><div><br></div></div></div></blockquote><div><br></div>Should all be in r160902.</div><div><br></div><div>- thanks, fariborz</div><div><br><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Jordan</div></blockquote></div><br></body></html>