Review Request: Move "Jump To File" functionality from the model of executescript-Plugin to the general OutputView model. (Thus enabling it for the execute and externalscript plugin's too.)

[Milian Wolff]

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


closing as discarded, esp. with a new patchset coming up: https://git.reviewboard.kde.org/r/104814/

- Milian Wolff


On Nov. 16, 2011, 7:54 p.m., Manuel Riecke wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103151/
> -----------------------------------------------------------
> 
> (Updated Nov. 16, 2011, 7:54 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> Move the functionality to detect filename+line number from the output of a executed script/exectuable from ExecuteScriptOutputModel to OutputModel. This adds this functionality to all plugins showing the output of a executable, e.g. execute, executescript and externalscript.
> 
> 
> Diffs
> -----
> 
>   outputview/outputmodel.h b4c9631 
>   outputview/outputmodel.cpp 317e3ee 
>   plugins/executescript/CMakeLists.txt 5d6168f 
>   plugins/executescript/executescriptoutputmodel.h 180adbd 
>   plugins/executescript/executescriptoutputmodel.cpp 1c852e9 
>   plugins/executescript/scriptappjob.cpp 3b68ca5 
> 
> Diff: http://git.reviewboard.kde.org/r/103151/diff/
> 
> 
> Testing
> -------
> 
> It still compiles, all the three plugins (execute, executescript and externalscript) work and have the "Jump to file" functionality.
> 
> 
> Thanks,
> 
> Manuel Riecke
> 
>

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