KDevelop Plugin for clang-tidy

Kevin Funk kfunk at kde.org
Fri Oct 7 11:13:24 UTC 2016


On Friday, 7 October 2016 07:57:37 CEST Carlos Nihelton Santana de Oliveira 
wrote:
> I'm sorry, guys, for having not answered you before. I think I couldn't
> push to any KDE repositories until yesterday. But I had no time to try.
> Thanks for your support, Kevin!

No worries, we're all busy :)

> I think I will have time after finishing my work day, since it will be
> weekend!

Sure, just make sure to rebase your work on what's in KDE's kdev-clang-
tidy.git, then push!

Cheers,
Kevin

> I'll let know!
> 
> Regards,
> Carlos
> 
> Em 7 de out de 2016 6:09 AM, "Kevin Funk" <kfunk at kde.org> escreveu:
> 
> On Wednesday, 21 September 2016 10:02:13 CEST Kevin Funk wrote:
> > On Tuesday, 20 September 2016 12:45:52 CEST Carlos Nihelton Santana de
> > 
> > Oliveira wrote:
> > > Thanks guys!
> > > 
> > > I think I found the way. It's just a matter of overriding
> 
> configPageType()
> 
> > > member function to return ConfigPage::AnalyzersPage.  I suggested that
> 
> to
> 
> > > kdev-cppcheck too.
> > > 
> > > Since this plugin is outside of KDevelop and KDevPlatform, how do I have
> > > to
> > > proceed to ask for review on Phabricator?
> > 
> > Heya Carlos.
> > 
> > I'll ask our sysadmins to create a new playground repo for
> 
> kdev-clang-tidy.
> 
> FYI: Repository created and populated with the current state of the Github
> repository.
> 
> @Carlos: Please continue to work on the KDE repository.
> 
> Thanks!
> Kevin
> 
> > In the meantime, you can still upload a full diff of your kdev-clang-tidy
> > repository to Phabricator. You can just upload a patch file; a
> 
> Differential
> 
> > in Phab doesn't need to have a repository associated afaik.
> > 
> > Hope that helps,
> > Kevin
> > 
> > > Rgds,
> > > Carlos
> > > 
> > > Em 20 de set de 2016 09:57, "Kevin Funk" <kfunk at kde.org> escreveu:
> > > > On Tuesday, 20 September 2016 14:41:39 CEST Milian Wolff wrote:
> > > > > On Sonntag, 18. September 2016 13:03:25 CEST Carlos Nihelton wrote:
> > > > > > Hi all,
> > > > > > 
> > > > > > Amazing things you all have done into KDevelop! It's getting more
> > > > > > and
> > > > > > more useful every day.
> > > > > > 
> > > > > > I read about the Problem Checker Framework and started to write a
> > > > 
> > > > plugin
> > > > 
> > > > > > for clang-tidy (https://github.com/CarlosNihelton/kdev-clang-tidy
> 
> ).
> 
> > > > > > It
> > > > > > is already in a useful state, but I plan to add support for
> > > > > > interactive
> > > > > > fixing code based on clang-tidy suggestions and
> > > > 
> > > > clang-apply-replacements
> > > > 
> > > > > > automatic fix feature.
> > > > > 
> > > > > Cool, sounds very useful!
> > > > 
> > > > That indeed sounds awesome!
> > > > 
> > > > Will try to review your code as soon as possible. Thanks for working
> 
> on
> 
> > > > this!
> > > > 
> > > > Cheers,
> > > > Kevin
> > > > 
> > > > > > When I updated my local repository of KDevelop I noticed a new
> > > > > > session
> > > > > > configuration page called Analyzers. How can I insert the
> 
> clang-tidy
> 
> > > > > > plugin into there? (Also I believe kdev-cppcheck will soon be
> > > > > > inserted
> > > > > > there too).
> > > > > 
> > > > > Anton (CC'ed), can you clarify how to do that?
> > > > > 
> > > > > > Also, I'd like to submit my plugin for review soon after that.
> > > > > 
> > > > > Looking forward to that.
> > > > > 
> > > > > Cheers
> > > > 
> > > > --
> > > > Kevin Funk | kfunk at kde.org | http://kfunk.org
> 
> --
> Kevin Funk | kfunk at kde.org | http://kfunk.org


-- 
Kevin Funk | kfunk at kde.org | http://kfunk.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 163 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20161007/b850090f/attachment.sig>


More information about the KDevelop-devel mailing list