Review Request: BuilderJob: (eye-candy) flatten composite job hierarchies and generate a meaningful job name

Ivan Shapovalov intelfx100 at gmail.com
Mon Sep 17 07:51:26 UTC 2012


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

Review request for KDevelop.


Description
-------

1. If a BuilderJob is added into another BuilderJob, the latter's subjobs are inserted into the former rather than having two nested composite jobs.
2. A BuilderJob has its name generated from its contents (this is real eye-candy).


Diffs
-----

  plugins/execute/executeplugin.cpp 817c082 
  plugins/projectmanagerview/projectmanagerviewplugin.cpp 968376d 
  project/builderjob.h 297f187 
  project/builderjob.cpp 44519d3 

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


Testing
-------

Existing unit-tests, various hand-testing.


Thanks,

Ivan Shapovalov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120917/67c5304d/attachment.html>


More information about the KDevelop-devel mailing list