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

List:       kde-commits
Subject:    Re: kdelibs/corba/cuteidl
From:       Lars Knoll <Lars.Knoll () mpi-hd ! mpg ! de>
Date:       1999-09-20 23:57:55
[Download RAW message or body]

On Tue, 21 Sep 1999, David Faure wrote:

> On Mon, Sep 20, 1999 at 11:23:56PM +0000, David Faure wrote:
> > On Tue, Sep 21, 1999 at 12:18:43AM +0200, Lars Knoll wrote:
> > > On Mon, 20 Sep 1999, CVS by dfaure wrote:
> > > 
> > > > 
> > > > kdelibs/corba/cuteidl codegen-qt-common.cc,1.4,1.5
> > > > Author: dfaure
> > > > CVSROOT: /home/kde
> > > > Mon Sep 20 23:58:22 MET DST 1999
> > > > Update of /home/kde/kdelibs/corba/cuteidl
> > > > In directory zeus:/tmp/cvs-serv710
> > > > 
> > > > Modified Files:
> > > > 	codegen-qt-common.cc 
> > > > Log Message:
> > > > Fixed typedefs for string and wstring. Great news : my kded compiles with cuteidl !
> > > > 
> > > Great to hear! Does it work too??? ;-)
> > 
> > Yes !
> > Would you believe it ? :)))

:-))) I wouldn't have thought that we get it working that fast...
> 
> I've started on kom.
> It's unbelievable the great simplifications this allows.
> The best example :
> 
> -KOM::Base::RelativesSeq* KOMBase::describeRelatives()
> -{
> -  KOM::Base::RelativesSeq* seq = new KOM::Base::RelativesSeq;
> -  int i = 0;
> -  seq->length( i );
> -
> -  QValueList<KOM::Base_var>::Iterator it = m_lstRelatives.begin();
> -  for( ; it != m_lstRelatives.end(); ++it )
> -  {
> -    seq->length( i + 1 );
> -    (*seq)[i++] = *it;
> -  }
> -
> -  return seq;
>  }  
> 
> All removed, and turned into the inline
>  KOM::Base::RelativesSeq describeRelatives() { return m_lstRelatives; }  

Wow... 
> 
> I love it !
> 
> We win on simplicity (for non-CORBA gurus), on speed (a lot of conversions removed)
> and on memory usage (useless code removed, and less data necessary : no conversion = sharing 
> the same object instead of copying/converting)
> 
And we win, because it's easier to verify, that there are no errors
inside...

Bye & good night,
Lars

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