D8317: Don't try to guess the language from given arguments

Milian Wolff noreply at phabricator.kde.org
Mon Oct 16 08:21:35 UTC 2017


mwolff accepted this revision.
mwolff added a comment.
This revision is now accepted and ready to land.


  lgtm

INLINE COMMENTS

> apol wrote in test_compilerprovider.cpp:127
> why did you remove 1 of de dashes?

both `man clang` and `man gcc` use this form, and above we also use this form. So I think this change is fine, but should go to a separate patch (commit it directly)

REPOSITORY
  R32 KDevelop

BRANCH
  dont-guess-language

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

To: aaronpuchert, #kdevelop, mwolff
Cc: mwolff, apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171016/b8617801/attachment.html>


More information about the KDevelop-devel mailing list