Review Request: Fix bug 308846 Clicking on messages does not move the cursor to that location.

Andreas Pakulat apaku at gmx.de
Wed Dec 12 22:03:59 UTC 2012


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



outputview/outputfilteringstrategies.cpp
<http://git.reviewboard.kde.org/r/107683/#comment17996>

    Why did you change the type-check here?


- Andreas Pakulat


On Dec. 12, 2012, 9:22 p.m., Morten Volden wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107683/
> -----------------------------------------------------------
> 
> (Updated Dec. 12, 2012, 9:22 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> Fix regression introduced when refactoring the filtering of build output. The regression meant that lines that had passed one of the regex (and were clickable) were made to be not-clickable.
> 
> This patch restores the behavior from before the refactoring, plus it makes it visible to the user that those lines are clickable. This should fix the remaining part of Bug 308846.
> 
> 
> This addresses bug 308846.
>     http://bugs.kde.org/show_bug.cgi?id=308846
> 
> 
> Diffs
> -----
> 
>   outputview/outputfilteringstrategies.cpp 567bc62 
>   outputview/tests/filteringstrategytest.cpp 116b163 
>   outputview/tests/testlinebuilderfunctions.h 17ec90d 
> 
> Diff: http://git.reviewboard.kde.org/r/107683/diff/
> 
> 
> Testing
> -------
> 
> Extended existing tests in outputfilteringstrategy
> 
> 
> Thanks,
> 
> Morten Volden
> 
>

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


More information about the KDevelop-devel mailing list