Review Request: Share code between makebuilder and external scripts

[Morten Volden]

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

(Updated May 6, 2012, 4:10 p.m.)


Review request for KDevelop.


Description
-------

So far I have:
   * Moved some of the filtering code
       - More specifically some of the code for filtering (Compiler output, script output ) 

   * Extended the outputmodel 
         - It now inherits from QAbstractListModel and IOutputview model
         - It is now possible to choose a filtering strategy on the outputview outputmodel
   * Plus a few other things to make things work

   * Started doing some Unit test of the different filters

The solution currently has four filtering strategies:

* No filter
* Compiler filter 
* Script Error flter 
* StaticAnalysis Filter (for cpp check, krazy, etc)

I think I have now come to a point where it would be good to get some feedback on the proposed solution

Work identified that remains to be done (Input is welcome):

* Finish tests for filter strategies
* Move MakeoutputModel tests to outputview and extend them
* Make Makebuildder use the code in outputview. 
* Make NofilteringStrategy faster (i.e. do not create a filtered item for each line)
* Make custom buildsystem use the code in outputview - Done (But Needs to be reviewed)
* Extend filterstrategies with other filters?? (Input on that one is most welcome)
* Discuss how the filters should be selected via the GUI (and in which use-cases it makes sense to do so)


Diffs (updated)
-----

  outputview/CMakeLists.txt 5d9e539 
  outputview/delegateholder.h PRE-CREATION 
  outputview/delegateholder.cpp PRE-CREATION 
  outputview/filtereditem.h PRE-CREATION 
  outputview/filtereditem.cpp PRE-CREATION 
  outputview/ifilterstrategy.h PRE-CREATION 
  outputview/ifilterstrategy.cpp PRE-CREATION 
  outputview/outputdelegate.h PRE-CREATION 
  outputview/outputdelegate.cpp PRE-CREATION 
  outputview/outputfilteringstrategies.h PRE-CREATION 
  outputview/outputfilteringstrategies.cpp PRE-CREATION 
  outputview/outputformats.h PRE-CREATION 
  outputview/outputformats.cpp PRE-CREATION 
  outputview/outputjob.h c17b592 
  outputview/outputjob.cpp 5b9db06 
  outputview/outputmodel.h b4c9631 
  outputview/outputmodel.cpp 317e3ee 
  outputview/tests/CMakeLists.txt PRE-CREATION 
  outputview/tests/filteringstrategytest.h PRE-CREATION 
  outputview/tests/filteringstrategytest.cpp PRE-CREATION 
  outputview/tests/outputviewtest.h PRE-CREATION 
  outputview/tests/outputviewtest.cpp PRE-CREATION 
  outputview/tests/projects/onefileproject/main.cpp PRE-CREATION 
  plugins/execute/nativeappjob.cpp 3eb654c 
  plugins/executescript/executescriptoutputmodel.h 180adbd 
  plugins/executescript/executescriptoutputmodel.cpp 1c852e9 
  plugins/executescript/executescriptplugin.h 4eea6a4 
  plugins/executescript/executescriptplugin.cpp e55b514 
  plugins/executescript/scriptappconfig.h 2e401ea 
  plugins/executescript/scriptappconfig.cpp d7ff984 
  plugins/executescript/scriptappjob.h 48400aa 
  plugins/executescript/scriptappjob.cpp 3b68ca5 
  plugins/externalscript/externalscriptjob.h 1922469 
  plugins/externalscript/externalscriptjob.cpp aeb5a34 
  plugins/externalscript/externalscriptoutputmodel.cpp 7a3d5d4 
  plugins/externalscript/externalscriptplugin.h c87aaa8 
  plugins/externalscript/externalscriptplugin.cpp 6d62b63 
  vcs/dvcs/dvcsjob.cpp 1e0f187 

Diff: http://git.reviewboard.kde.org/r/104814/diff/


Testing
-------

Started working on Unittest for filtering strategies. Plan to move most tests from Makeoutputmodel at some point. Plus used the patch for a week now and nothing seemingly breaks...


Screenshots
-----------

CPPCheckBefore
  http://git.reviewboard.kde.org/r/104814/s/550/
CppcheckAfter
  http://git.reviewboard.kde.org/r/104814/s/551/
QuickCompileBefore
  http://git.reviewboard.kde.org/r/104814/s/552/
QuickCompileAfter
  http://git.reviewboard.kde.org/r/104814/s/553/


Thanks,

Morten Volden

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120506/24b08ad6/attachment.html>