From kde-commits Thu Dec 18 16:53:33 2003 From: George Staikos Date: Thu, 18 Dec 2003 16:53:33 +0000 To: kde-commits Subject: Re: kdelibs/kwallet/client X-MARC-Message: https://marc.info/?l=kde-commits&m=107176651307315 On Thursday 18 December 2003 09:34, Christian Loose wrote: > > > 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 > > It actually already does something similar. You can let dcopidl2cpp create > a DCOPStub. Then you have a class that does all the ugly stuff of calling a > DCOP method. So instead of writing > > DCOPRef("kded", "kwalletd").call("folderDoesNotExist", wallet, folder); > > you do > > Backend_stub("kded", "kwalletd").folderDoesNotExist(wallet, folder); > > and the compiler checks for you if you have a typo. Which still leaves two typos that it doesn't catch. Also I didn't mention it, but in this case we also have a circular dependency since kwalletd uses the kwallet library, and kwallet library calls kded. -- George Staikos KDE Developer http://www.kde.org/ Staikos Computing Services Inc. http://www.staikos.net/