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

List:       kdevelop-devel
Subject:    Re: KDE/kdevplatform
From:       David Nolden <zwabel () googlemail ! com>
Date:       2009-11-26 22:51:49
Message-ID: 200911262351.49746.zwabel () googlemail ! com
[Download RAW message or body]

Am Donnerstag 26 November 2009 20:02:53 schrieb Andreas Pakulat:
> > The problem is that by removing this feature wholly, you introduce new
> > bugs in the code-generation.
> 
> Huh? Are you kidding? There's exactly 2 lines of source code that I
> touched in code-generation and they were already "optional" if no
> formatter was available.

The problem is the follow-up problems that result from the source-formatters 
not being there. I have written some code that simply _assumes_ that there is 
a source-formatter, else it won't even _indent_ inserted code properly.

Such crappy source-code insertion is simply not what I want, I've put too much 
work into that stuff to let it be devaluated so senselessly. If you would have 
removed the source-formatters earlier, then I would have screemed earlier, and 
we could have found a viable soloution.

Anyway getting to the point: We _need_ some kind of source-formatting for the 
C++ support.

The options are:
1. Hack down the existing source-formatter support, thereby omitting its few 
significant bugs
2. Derive a new minimalistic maintainable source-formatter plugin that has no 
UI at all, and just calls a command-line tool that is read from some place in 
the sources

I'm willing to do both of these things. Both of them are a reduction of the 
source-formatter support that existed when the feature-freeze was entered, and 
thus viable options.

The simplest option is probably option 1 for the 4.0 Release as it's well 
tested, and then unless the source-formatter gets some significant maintenance 
option 2 for 4.1.

Greetings, David

--
KDevelop-devel mailing list
KDevelop-devel@kdevelop.org
https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic