Skip to content

Set AllowAllArgumentsOnNextLine to false

Alexander Lohnau requested to merge work/formatting into master

Discussion in plasma/plasma-workspace!381 (comment 132887)

This is a bit more controversial, but in most cases it prevents uglification.

See a diff of pw, before this value was not set:
file.diff

The only alternative we would have is to exclude these lines from the formatting or to do it this way:

QDBusMessage msg = QDBusMessage::createMethodCall(
    QStringLiteral("org.kde.plasmashell"),//
    QStringLiteral("/org/kde/osdService"),//
    QStringLiteral("org.kde.osdService"),//
    QStringLiteral("showText")
);

But IMHO this does not look good.

@broulik @nicolasfella @cblack @cullmann @davidedmundson

Merge request reports