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

[Commit Hook]

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


This review has been submitted with commit b9c3ed8bd23bae2a711a31f8891da2abf7a24422 by Morten Danielsen Volden to branch master.

- Commit Hook


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/20121214/daf1412c/attachment.html>