Deprecate suggestName
ClosedPublic

Authored by nicolasfella on Jul 22 2019, 9:35 AM.

Details

Summary

KCoreAddons now has a drop-in replacement. Deprecate this so it can be removed in KF6

Test Plan

Buils. Deprecation warnings are thrown due to internal use that will be ported in a future patch

Diff Detail

Repository
R241 KIO
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
nicolasfella created this revision.Jul 22 2019, 9:35 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptJul 22 2019, 9:35 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
nicolasfella requested review of this revision.Jul 22 2019, 9:35 AM
dfaure added inline comments.Jul 22 2019, 3:23 PM
src/core/global.h
144

I like it when we say when something got deprecated, like this

  • @deprecated since 5.61, use KFileUtils::suggestName() from KCoreAddons
  • Include deprecated since
  • Don't build with KIOCORE_NO_DEPRECATED
dfaure accepted this revision.Jul 22 2019, 8:11 PM
This revision is now accepted and ready to land.Jul 22 2019, 8:11 PM
This revision was automatically updated to reflect the committed changes.