https://bugs.kde.org/show_bug.cgi?id=367966
--- Comment #6 from Bernhard Scheirle <bernhard+...@scheirle.de> ---
I added some debug outputs to the "C Style" indentation script and it looks
like the scripts `indent` function does not get called for the opening bracket
"{" if auto brackets are enabled.
Therefore the indentation level for the opening bracket does not get modified
to match the if statement.
In addition the auto bracket feature moves the cursor in between the two
brackets and than calls the `indent` function for the closing bracket "}". Now
the script does not find the opening bracket correctly (skips over it) since
the script expects the cursor to be behind the closing bracket.
The workaround below manually processes the opening bracket instead of closing
one if "{|}" (| = cursor position) is in the current line.
It seems to work quite well, but a proper fix would be nice.
diff --git a/ktexteditor/src/script/data/indentation/cstyle.js
b~/.local/share/katepart5/script/indentation/mycustomindentation.js
index 86e72a7..5916506 100644
--- a/ktexteditor/src/script/data/indentation/cstyle.js
+++ b~/.local/share/katepart5/script/indentation/mycustomindentation.js
@@ -1,5 +1,5 @@
var katescript = {
- "name": "C Style",
+ "name": "C Style Modified",
"author": "Dominik Haumann <dh...@gmx.de>, Milian Wolff
<m...@milianw.de>",
"license": "LGPL",
"revision": 4,
@@ -772,7 +772,11 @@ function processChar(line, c)
return filler;
} else if (firstPos == column - 1 && c == '}') {
- var indentation = findLeftBrace(line, firstPos);
+ if (document.charAt(cursor) == '}' && document.charAt(cursor.line,
cursor.column-1) == '{')
+ {
+ return processChar(line, '{')
+ }
+ var indentation = findLeftBrace(line, firstPos)
if (indentation == -1)
indentation = -2;
return indentation;
--
You are receiving this mail because:
You are watching all bug changes.
