[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