Skip to content

Use new version-controlled enumerator deprecation warning macros

Friedrich W. H. Kossebau requested to merge work/kossebau/useenumeratordepwarnings into master

Usage example for extra-cmake-modules!115 (merged)

Sadly clang-format here gets into the game, where I would have liked the style

    SaveOptions ///< @deprecated Since 5.38, no known users.
    KCONFIGWIDGETS_ENUMERATOR_DEPRECATED_VERSION_BELATED(5, 82, 5, 38, "No known users"),

clang-format wants the second line with another level of indentation:

    SaveOptions ///< @deprecated Since 5.38, no known users.
        KCONFIGWIDGETS_ENUMERATOR_DEPRECATED_VERSION_BELATED(5, 82, 5, 38, "No known users"),

Any ideas what to do here? Or other layouts? Personally I would fancy to have the enumerator and the API dox on the same line as before and as also done for all the other enumerators for consistency, and have the macro on its own line, as done in the patch, that seems most simple to quickly parse with human eyes.

The macro has to be placed between the enumerator and the "," in any case.

Merge request reports