D24684: Use ECMGenerateExportHeader to manage deprecated API better

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Fri Oct 18 15:16:09 BST 2019


kossebau added a comment.


  In D24684#549785 <https://phabricator.kde.org/D24684#549785>, @dfaure wrote:
  
  > The commit log says KIOKIOFILEWIDGETS with KIO duplicated, that's a typo in the commit log, but only there, right?
  
  
  Typo indeed, fixed.

INLINE COMMENTS

> dfaure wrote in global.cpp:99
> Isn't some ifdef still needed for this deprecated method?

I forgot to explicitly mention this in the commit message: no support for EXCLUDE_DEPRECATED_BEFORE_AND_AT for KIO here yet, as version-level-based disabling of API needs some work first, as some undeprecated API is implemented in terms of deprecated still, as I found when testing that support.
The currently existing `#ifndef KIOCORE_NO_DEPRECATED` in the source files thus got only removed.

Gzess that should get an explicit TODO comment, e.g. near the ecm_generate_export_header marcos?

> dfaure wrote in kfileitem.cpp:1242
> I'm confused, isn't this what *_BUILD_DEPRECATED_SINCE is for?
> 
> [repeats]

See other comment.

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D24684

To: kossebau, #frameworks, dfaure
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191018/f9a177a4/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list