[bug][patch] languages/cpp/problemreporter

Matt Rogers mattr at kde.org
Tue Oct 31 00:52:40 UTC 2006


On Monday 30 October 2006 09:57, Stephan Diederich wrote:
> Hi there,
>
> the attached patch fixes a bug in the cpp-problemreporter.
> Problem was that the path of out of project-root source-files was not
> properly assembled.
>
> Old version:
>   QString relFileName = filename;
>   relFileName.remove(m_cppSupport->project()->projectDirectory());
> New version:
>   QString relFileName =
> m_cppSupport->project()->relativeProjectFile(filename);
>
> This was done several times. It caused an additional "/" to add during
> assembling of the final path.
>
> Patch tested with automake projects (kdevelop.kdevelop) and qmake-projects.
>
> Totally unrelated: changed the single occurrence kdDebug(9008) to
> kdDebug(9007).
>
> Can this be checked and applied before release of 3.4?
>
> HTH,
> Stephan

Patch committed. Thanks!

Please do not bzip patches unless they're larger than 30KB in 
size before being compressed. It makes them harder to apply quickly, and I'm 
all about the quick (since otherwise, i get lazy, and forget to commit) :)
-- 
Matt




More information about the KDevelop-devel mailing list