https://bugs.kde.org/show_bug.cgi?id=441603

Jan Paul Batrina <jpmbatrin...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jpmbatrin...@gmail.com

--- Comment #1 from Jan Paul Batrina <jpmbatrin...@gmail.com> ---
Created attachment 141241
  --> https://bugs.kde.org/attachment.cgi?id=141241&action=edit
Hacky ruby.js diff which fixes the issue

Hello, thanks for the bug report + analysis!

The rules for indentation are written in javascript. Particularly, the behavior
you describe is due to
https://invent.kde.org/frameworks/ktexteditor/-/blob/master/src/script/data/indentation/ruby.js#L335
since the closing / in the regex is being treated as an incomplete division
operator (see
https://invent.kde.org/frameworks/ktexteditor/-/blob/master/src/script/data/indentation/ruby.js#L113).

git blame shows that both lines have been unchanged since the beginning of the
git repo. Do you remember the specific version where it previously worked?
Maybe there were Debian-specific patches for this?

The attached diff fixes the issue, but it relies on the syntax highlighting
enging, which is a very hacky way especially if we need to consider other
filetypes that use the ruby indentation style. If you find a cleaner way, feel
free to submit an MR to
https://invent.kde.org/frameworks/ktexteditor/-/merge_requests !

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to