[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-commits
Subject: Re: kdesdk/cervisia
From: George Staikos <staikos () kde ! org>
Date: 2003-10-28 9:01:43
[Download RAW message or body]
On Tuesday 28 October 2003 03:50, Christian Loose wrote:
> I know, that we had this discussion before, but don't you think that this
> is a "fight against windmills"?
Well KDE is supposed to be portable. Let's keep it that way.
> Looking at lxr.kde.org (search for #std..map#) there is more and more stl
> code creeping into KDE's code base (ksvg, umbrello, taglib,...). So maybe
> we need a general rule for the usage of the STL?
I don't see why STL has to be used when Qt is sufficient and certainly
portable. It's also far more clear to developers when Qt containers are
used. You know what to expect because you deal with it regularily. We have
absolute nightmares of code in CVS right now, written in STL or glib, and
no-one wants to touch them. That's not to say that we don't have bad Qt code
too, but at least it's a bit more pleasant to fix. FYI I have seen symbols
come out of the STL code in KDE CVS that were actually thousands of
characters long. It's a nightmare to deal with STL, really. Let's avoid it
where we can.
Hm on that note, let's try to avoid starting this thread over again too.
Basically I'm just going to keep changing STL code over to Qt code as I
encounter it. It's easy to do, and it makes porting easy. I'm sure there
are some cases where STL is necessary, but these are very rare and I can't
remember seeing one in CVS yet. Actually much of the STL code in CVS tends
to be horribly buggy, as the ones who demand to use STL instead of Qt
containers don't actually understand the STL properly anyways. Restarting
this whole debate is the biggest waste of time though. I won't get dragged
into it this time.
> BTW did you try to use stlport on AIX?
No I'm using xlC as provided by IBM. Playing games like that is a waste of
time.
--
George Staikos
KDE Developer http://www.kde.org/
Staikos Computing Services Inc. http://www.staikos.net/
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic