Review Request 117695: change where dynamic replace tabs is performed

Matthew Woehlke mw_triad at users.sourceforge.net
Tue Apr 22 21:13:51 UTC 2014


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


If the option to reindent pasted code is enabled, does *that* still remove tabs? (Seems like you'd want it too in that case...)

- Matthew Woehlke


On April 22, 2014, 5:06 p.m., Sven Brauch wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117695/
> -----------------------------------------------------------
> 
> (Updated April 22, 2014, 5:06 p.m.)
> 
> 
> Review request for KDE Frameworks and Christoph Cullmann.
> 
> 
> Repository: ktexteditor
> 
> 
> Description
> -------
> 
> This makes typeChars handle replacing tabs by spaces, instead of insertText. The rationale is that insertText is often called programatically, and the caller should be able to rely on the text he requests to be inserted is actually inserted, and not changed on-the-fly. Examples for where the previous solution caused problems are KDevelop (the codegen) and kte-collaborative.
> 
> I'm not sure what the code I removed was doing (heh). It looks like it is supposed to advance to the next indent level if the current indent level is odd, but that still works after removing it.
> 
> The obvious user-visible change here is that tabs in pasted text will no longer be replaced. But since I always found this behaviour undesirable anyways, I did not bother to replicate it. I will instead wait for people to yell at me for removing it. ;)
> 
> 
> Diffs
> -----
> 
>   src/document/katedocument.cpp 546d3e6aadc57f24c3fa766ce235addc0f02e3c3 
> 
> Diff: https://git.reviewboard.kde.org/r/117695/diff/
> 
> 
> Testing
> -------
> 
> Just some quick manual tests, it seems to still work as intended.
> 
> 
> Thanks,
> 
> Sven Brauch
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140422/22e90320/attachment.html>


More information about the Kde-frameworks-devel mailing list