Review Request: Share code between makebuilder and external scripts

Morten Volden mvolden2 at
Fri Jul 20 22:22:43 UTC 2012

This is an automatically generated e-mail. To reply, visit:

(Updated July 20, 2012, 10:22 p.m.)

Review request for KDevelop.


Added an ErrorFormat for Static analysis (krazy2) updated Unittests


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/outputmodeltest.h PRE-CREATION 
  outputview/tests/outputmodeltest.cpp PRE-CREATION 
  outputview/tests/projects/onefileproject/main.cpp PRE-CREATION 
  outputview/tests/testlinebuilderfunctions.h 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/iexecutescriptplugin.h 3c31d3c 
  plugins/executescript/scriptappconfig.h 2e401ea 
  plugins/executescript/scriptappconfig.cpp d7ff984 
  plugins/executescript/scriptappconfig.ui ebc1e9e 
  plugins/executescript/scriptappjob.h 48400aa 
  plugins/executescript/scriptappjob.cpp 3b68ca5 
  plugins/externalscript/editexternalscript.cpp ef1b03f 
  plugins/externalscript/editexternalscript.ui cd747e6 
  plugins/externalscript/externalscriptitem.h 9209ad2 
  plugins/externalscript/externalscriptitem.cpp 113845f 
  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 



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...




Morten Volden

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the KDevelop-devel mailing list