<table><tr><td style="">kossebau added a comment.
</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/D25401">View Revision</a></tr></table><br /><div><div><p>One thing I saw with the Qt deprecation tags is that deprecation attributes were only added to constructor calls, not the class itself.<br />
Which to me made some sense, as one should be warned when creating instances of that class. But once you are passed an instance due to other reasons, being warned about furher calls on the instance makes no real sense, as one has to use the instance now that it exists.<br />
In the initial set of patches to KF with the new deprecation macros I also did it like that for deprecated classes,  added the warning macros only to constructor or other generation functions, not next to the class keyword.</p>

<p>Not sure how compilers actually react on deprecation attributes to the class only, still have to look up in what warnings on which class usages that results.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25401">https://phabricator.kde.org/D25401</a></div></div><br /><div><strong>To: </strong>dfaure, vkrause, kossebau<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>