[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kwrite-devel
Subject:    [Bug 274030] New: Code folding should be more clever about
From:       Rolf Eike Beer <kde () opensource ! sf-tec ! de>
Date:       2011-05-24 13:19:24
Message-ID: bug-274030-4792 () http ! bugs ! kde ! org/
[Download RAW message or body]

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

           Summary: Code folding should be more clever about preprocessor
                    defines
           Product: kate
           Version: 0.2
          Platform: openSUSE RPMs
        OS/Version: Linux
            Status: NEW
          Severity: wishlist
          Priority: NOR
         Component: folding
        AssignedTo: kwrite-devel@kde.org
        ReportedBy: kde@opensource.sf-tec.de


Version:           0.2 (using KDE 4.6.3) 
OS:                Linux

This is basically a resurrection of bug 265225 for kate. The basic problem is:

-- quote --
If I have a preprocessor #if/#else/#endif and open a code block (like "if ()
{") in both of them that is closed outside the preprocessor block the folding
logic doesn't take the preprocessor block into account, i.e. it ignores the
fact that only _one_ of those blocks can actually be in use. That leads to the
fact that the folder miscounts the number of braces and folds completely wrong.
-- endquote --

So even if katepart doesn't know about which #ifdef block will be active it
should count the opening/closing braces in only one of them. This could still
be bad if it chooses the "wrong" one but this would still be much better than
the current situation where the folding for the rest of the file gets totally
screwed.

Reproducible: Always




OS: Linux (i686) release 2.6.37-12-desktop
Compiler: gcc

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
KWrite-Devel mailing list
KWrite-Devel@kde.org
https://mail.kde.org/mailman/listinfo/kwrite-devel
[prev in list] [next in list] [prev in thread] [next in thread]