♟ kfunk

Is someone willing to pick up this patch again? It's still an issue.

Due to https://blog.qt.io/blog/2018/10/29/deprecation-of-qbs/ this should probably be abandoned.

run-clang-tidy: Use check_call instead of check_output

Marking this Diff properly -- it needs more work.

Bump? This is a trivial one

In D37479#861580, @Eugene.Zelenko wrote:

I think will be good idea to introduce command line option to control progress display: turn it off, show in absolute numbers or percents.

Make run-clang-tidy compatible with Python 3.x

[clang-tidy] Fixup clang-apply-replacements/invalid-files test

Seems to have worked:

Committing to https://llvm.org/svn/llvm-project/clang-tools-extra/trunk ...
        A       test/clang-apply-replacements/Inputs/invalid-files/invalid-files.yaml
        A       test/clang-apply-replacements/invalid-files.cpp
        M       clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp
Committed r308974

[clang-tidy] clang-apply-replacements: Don't insert null entry

Remove unnecessary sed line in test driver

Looks good from my side. Can push if it gets another +1.

@erikjv Ready for review? Does this reliably fix https://bugreports.qt.io/browse/QTCREATORBUG-15449?

Add clazy to external Clang examples page

Fix for clang_Cursor_getSpellingNameRange()

@other LLVM devs: Please review so we can finally ship this?

In D21279#455923, @klimek wrote:

Generally, please subscribe cfe-commits when sending patches via phab.
See http://llvm.org/docs/Phabricator.html

Remove unrelated hunk

@bkramer: Thoughts about my patch? I wonder if this makes your "work-around" in D17808 redundant.

Committed. Will use "Differential Revision: <id>" next time, forgot to amend...

Bump libclang API version after r262318

Okay, just gave this another look. Unit test seems fine as-is. Let's call loadPlugins there explicitly.

Remove TODO statement

Note: Unit test is still not functional. I'll have another look at it now.

Bump? Waiting for another +1 here.

Note: I don't have commit rights yet

@realincubus: Sorry, I didn't know you had put this up for review already.

Just uploading my WIP patch, now that the branching comes close. I added a test, unfortunately it doesn't do what I expect.
Please see the FIXME.

By the way; prove that it really works: http://wstaw.org/m/2015/12/23/kdevelop-clazy.png (loving it!) :)

Doing as requested by http://lists.llvm.org/pipermail/cfe-dev/2014-October/039381.html, let's put this up for review in its initial form, so we can discuss a better approach.

Looks good to me now. I'd also appreciate another +1 from somone else, though.

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

kfunk (Kevin Funk)
User

Projects

User Details

Recent Activity
View All

Apr 7 2021

Nov 13 2019

Feb 25 2019

Nov 27 2017

Sep 20 2017

Sep 5 2017

Aug 25 2017

Jul 25 2017

Jul 11 2017

Jul 10 2017

Apr 25 2017

Feb 28 2017

Feb 24 2017

Feb 22 2017

Dec 20 2016

Dec 15 2016

Dec 14 2016

Nov 3 2016

Sep 15 2016

Jun 13 2016

Mar 24 2016

Mar 15 2016

Mar 10 2016

Mar 8 2016

Mar 7 2016

Mar 5 2016

Mar 3 2016

Mar 2 2016

Feb 21 2016

Jan 12 2016

Jan 11 2016

Dec 24 2015

Dec 22 2015

Nov 24 2015

Nov 4 2015