<table><tr><td style="">shaheed added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D4510" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D4510#inline-18217" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">skelly</span> wrote in <span style="color: #4b4d51; font-weight: bold;">sip_generator.py:172</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">It was possible to handle exports without looking for the text EXPORT in the MACRO NAME. Why is deprecated different?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Because the expansion of the attribute in this case contains not a string, but a compiler reserved value</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">__deprecated__</pre></div>

<p style="padding: 0; margin: 8px;">in the case of clang. I am not aware of a portable way to know what the expansion is, so the choice was between a string compare of the clang spelling, or the original. The latter seemed somewhat more reliable to me. IIRC, my original code did the same for the EXPORT stuff (I know you changed it and to be honest, I think that was suboptimal for the same reason, but at least the text appears as "hidden" not an opaque reserved value).</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4510" rel="noreferrer">https://phabricator.kde.org/D4510</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>shaheed, skelly, Build System, Frameworks<br /><strong>Cc: </strong>Frameworks, Build System<br /></div>