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