-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120443/#review68224
-----------------------------------------------------------


Hi, I think that should not be in KTextEditor.
Thats really special and given that only a very limited amount of languages 
with ctags specials exists, it would make more sense to just have a heuristic 
in the project plugin to figure that out on its own.

- Christoph Cullmann


On Sept. 30, 2014, 8:34 p.m., Bastian Holst wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120443/
> -----------------------------------------------------------
> 
> (Updated Sept. 30, 2014, 8:34 p.m.)
> 
> 
> Review request for KDE Frameworks and Christoph Cullmann.
> 
> 
> Repository: ktexteditor
> 
> 
> Description
> -------
> 
> Add ctags configuration option to highlighting
> 
> I need the added function to ask for the ctags command from kate's project 
> plugin. I am still not sure if this is the right place to add the function. 
> What do you think?
> 
> 
> Diffs
> -----
> 
>   src/document/katedocument.cpp 82363ed73bac515918b99d935cfc53d667bde3f7 
>   src/include/ktexteditor/mainwindow.h 
> bfee68a6536282d4ac58fc6adf16349b0d514d0b 
>   src/mode/katemodemanager.h e249913483cd184e9c66066bc49907fd49325a9d 
>   src/mode/katemodemanager.cpp 694ca6219c80b8812f86912e58f3c14fe9324f92 
>   src/syntax/data/haskell.xml f0e7f8f955ea9cd56795add8de03c6627313301a 
>   src/syntax/data/katehighlightingindexer.cpp 
> 3f02e8c2a76e471489bc6df2a9f4ac0463c3afe0 
>   src/syntax/data/language.xsd 3f20304efe9c7a29df0dbe0a822196ec612437e8 
>   src/syntax/data/literate-haskell.xml 
> f69dbc56d7effbfa02398456cea616f0fce2d706 
>   src/syntax/katehighlight.h 237cf1a1837a4cf7d530be5a6a74ba9df7d0f8b7 
>   src/syntax/katesyntaxmanager.h 94c2b597cb0935feea01e454ba92eddc24be7af6 
>   src/syntax/katesyntaxmanager.cpp c51fbaf9eb230dfd1fb1602000860bf21a5cd8ce 
>   src/utils/mainwindow.cpp 6e664a9e7d0d9301ef12d9b1d7050fa2e3007d6f 
> 
> Diff: https://git.reviewboard.kde.org/r/120443/diff/
> 
> 
> Testing
> -------
> 
> Testing with modified version of Kates project plugin 
> (https://git.reviewboard.kde.org/r/120444/).
> 
> 
> Thanks,
> 
> Bastian Holst
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to