Review Request: Fix MakeOutput items for cmake builds with relative paths

[Manuel Massing]

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

(Updated Feb. 25, 2011, 6:17 p.m.)


Review request for KDevelop.


Changes
-------

Use hash table (via QSet) instead of checking the string list for containment, to avoid O(n^2) runtime behaviour.


Summary
-------

Fixes MakeOutput items for cmake builds which use relative paths 
(i.e. cmake projects with CMAKE_USE_RELATIVE_PATHS set to on).

Previously, errors generated by source files with relativ paths
were not clickable because MakeOutputModel does not know about
the directories they are built in.

Because cmake is not verbose enough to indicate directory changes,
observes where object files are being built and use these paths
as search path for urlToFile.


Diffs (updated)
-----

  projectbuilders/makebuilder/makeoutputmodel.h 5ac4553 
  projectbuilders/makebuilder/makeoutputmodel.cpp 09bb72e 

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


Testing
-------

Tested on cmake project with SET(CMAKE_USE_RELATIVE_PATHS ON).


Thanks,

Manuel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110225/7f0cedd5/attachment.html>