D17318: WIP: Code Analysis: Show file extentions where the tool runs on

[gregormi]

gregormi added inline comments.

INLINE COMMENTS

> dhaumann wrote in kateprojectcodeanalysistool.h:62
> Why not return a QStringList? The current version just forces parsing in other places. :)
> 
> ...
> 
> Later: ok, now I see it's used in the regular expression. Still, I don't like how an implementation detail in a specific location (here: regex later) imposes design decisions here. And also in the UI, a label would rather be comma separated instead of | separated.

I added a comment about this.

> dhaumann wrote in kateprojectinfoviewcodeanalysis.cpp:105
> Why are there leading spaces in the label? Usually, spacing is up to the QStyle.

Fixed by using a button which is much less obstrusive

REPOSITORY
  R40 Kate

BRANCH
  arcpatch-D17318

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

To: gregormi, #kate, cullmann
Cc: cullmann, dhaumann, ngraham, kwrite-devel, hase, michaelh, demsking, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20181208/15213441/attachment-0001.html>