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

Igor Kushnir <igor...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|REPORTED                    |CONFIRMED
                 CC|                            |igor...@gmail.com

--- Comment #1 from Igor Kushnir <igor...@gmail.com> ---
> The cause is likely the KDevelop::extractFormattedTextFromContext function 
> from kdevplatform/util/formattinghelpers.cpp file, called from 
> plugins/customscript/customscript_plugin.cpp . The text matching done there 
> fails to match parts of the file when its lines have been switched around.
You are right. My reimplementation of extractFormattedTextFromContext() in
https://invent.kde.org/kdevelop/kdevelop/-/merge_requests/118 was recently
merged and will be available in the next released KDevelop version (5.13 or
6.0). But the reimplementation does not support reordering include lines.

Since the include lines contain alphanumeric (significant) characters, a
separate specific mechanism to ignore such diff will have to be implemented.
The implementation should be very careful and limited to avoid breaking user
code by mismatching context(s) or text. If you have an idea how to fix this, we
can discuss it here or you can create a (Draft) merge request for a more
involved discussion and review.

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

Reply via email to