D6903: [Feature] Enable per-project setting of source formatters

[Friedrich W. H. Kossebau]

kossebau marked 4 inline comments as done.
kossebau added a comment.


  @kfunk Thanks for review!

INLINE COMMENTS

> kfunk wrote in sourceformatterselectionedit.h:2
> Minor: Indentation/style looks a bit odd in the license header.

Tried to keep changes small by doing a plain copy of original one.
 Perhaps given this is a new file, even if forked, might be a reason to apply usual style, doing that now.

> kfunk wrote in sourceformatterselectionedit.h:53
> In order to make this a proper pimpl'd class you need to remove the private symbols as well. Can be done in a separate commit though

Would do in separate commit. Actually when doing the d-pimpl cleanup I saw more of such private things in the API and learned I have no clue if this is okay or not. Will poke you/people on irc for some discussion the next days.

REPOSITORY
  R33 KDevPlatform

BRANCH
  perprojectformatterconfig

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

To: kossebau, #kdevelop, kfunk
Cc: kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170805/e1c84025/attachment.html>