Review Request 124411: Display issues in the problems toolview, instead of a separate dedicated toolview.

Laszlo Kis-Adam dfighter1985 at gmail.com
Mon Jul 20 23:20:42 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124411/
-----------------------------------------------------------

(Updated July 21, 2015, 1:20 a.m.)


Review request for KDevelop.


Changes
-------

While the parent cleans up the ProblemModel instance here, let's be gracious and clean up after ourselves.


Repository: kdev-cppcheck


Description
-------

Up until now the plugin had it's own models and view, and toolview where it showed the issues. This patch makes it show the issues in a dedicated tab of the problems toolview. As a side effect, lots of code is removed!
See the video:
https://www.youtube.com/watch?v=mcfr-Kc46Yo

NOTE:
The video shows more than the issues. It shows the entire new GUI of the plugin.
I've already pushed the settings related GUI changes, so the relevant part starts around 0:15 in the video.


Diffs (updated)
-----

  CMakeLists.txt 58a6a85 
  ChangeLog a5110f7 
  back/cppcheck_file_model.h 7ab8652 
  back/cppcheck_file_model.cpp 8b7d997 
  config/genericconfigpage.cpp 03dafbf 
  config/ui/genericconfig.ui 03d74d3 
  icppcheckitem.h b9a8c6f 
  imodel.h 6b873c7 
  imodel.cpp 70f18dd 
  iparser.h b5b136f 
  iview.h 24cf8ae 
  job.h abb0514 
  job.cpp 42b9c09 
  kcm_kdev_cppcheck.json 67c3053 
  marks.h 330813c 
  marks.cpp d5aaf3b 
  models/cppcheck_file_item.h c37f10f 
  models/cppcheck_file_item.cpp 80b16f4 
  models/cppcheck_file_model.h 566fabf 
  models/cppcheck_file_model.cpp be6fe74 
  models/cppcheck_severity_item.h 27d0242 
  models/cppcheck_severity_item.cpp 1ae2c42 
  models/cppcheck_severity_model.h 6feb92e 
  models/cppcheck_severity_model.cpp c33e163 
  models/cppcheckitems.h 537edd0 
  models/cppcheckitemsimpl.h cfc64dc 
  models/cppcheckitemsimpl.cpp 95162e7 
  models/cppcheckmodel.h 3551e2e 
  models/cppcheckmodel.cpp 60327a7 
  modelwrapper.h 6c18078 
  modelwrapper.cpp c568698 
  parsers/cppcheckparser.h 1ba5148 
  parsers/cppcheckparser.cpp 768986d 
  plugin.h 882768e 
  plugin.cpp 72030fb 
  views/cppcheckview.h 31f36f2 
  views/cppcheckview.cpp 7d1096f 
  widget.h b5dc31a 
  widget.cpp 971382f 

Diff: https://git.reviewboard.kde.org/r/124411/diff/


Testing
-------

* builds
* works as intended


Thanks,

Laszlo Kis-Adam

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150720/69d43892/attachment-0001.html>


More information about the KDevelop-devel mailing list