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

List:       kde-devel
Subject:    Re: kdelibs/kwallet/client
From:       Alexander Kellett <me () lypanov ! net>
Date:       2003-12-18 1:46:59
[Download RAW message or body]

On Thu, Dec 18, 2003 at 01:50:03AM +0100, Friedrich W. H.  Kossebau wrote:
> Am Mittwoch, 17. Dezember 2003 23:23 schrieb George Staikos:
> > On Wednesday 17 December 2003 17:02, Volker Krause wrote:
> > > CVS commit by vkrause:
> > >
> > > Fix a typo in folderDoesNotExist() causing it to always return true.
> > >
> > > Fixes #70425.
> > > --- kdelibs/kwallet/client/kwallet.cc  #1.40:1.41
> > > @@ -618,5 +618,5 @@ void Wallet::walletOpenResult(int id) {
> > >
> > >  bool Wallet::folderDoesNotExist(const QString& wallet, const QString&
> > > folder) { -DCOPReply r = DCOPRef("kded",
> > > "kwalletd").call("fodlerDoesNotExist", wallet, folder); +DCOPReply r =
> > > DCOPRef("kded", "kwalletd").call("folderDoesNotExist", wallet, folder);
> > > bool rc = true;
> >
> >   Wow nice catch!  Thanks!
> 
> Those mistakes make me wonder why noone seems to use 
> #define SomeIDString "important"
> or
> static const char const* SomeIDString = "Important";
> 
> Is there any reason for this?

or even better why not have dcopidl2cpp itself create these .h's

Alex
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]