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

List:       kde-commits
Subject:    Re: kdenetwork/ktalkd/ktalkd
From:       Lubos Lunak <l.lunak () suse ! cz>
Date:       2003-01-15 10:39:36
[Download RAW message or body]

On Tuesday 14 of January 2003 20:48, Dirk Mueller wrote:
> On Die, 14 Jan 2003, Lubos Lunak wrote:
> > > Nope it's the other way around. Please do not use NULL.
> >
> >  There's no good reason for telling other people not to use NULL.
>
> What about: it looks ugly ? :)

 Well, I then suggest we stop using symbolic names completely ;). Or you 
really think NULL is more ugly than a zillion overloaded cases of plain 0? 
You should try hacking kwin (X11) for a while.
 
>
> What about: many platforms do not #define NULL for C++, hence its
> unportable?

 I wonder how we manage to compile KDE on those systems then. Assuming KDE 
actually compiles on such system.

>
> I think the varags case is a special case where it should be 0L and not 0.

 That's not guaranteed to be correct.

>
> But it still makes no sense at all to use a preprocessor define!

 I'd be fine with a const variable too *shrug*.

-- 
Lubos Lunak
KDE developer
---------------------------------------------------------------------
SuSE CR, s.r.o.  e-mail: l.lunak@suse.cz , l.lunak@kde.org
Drahobejlova 27  tel: +420 2 9654 2373
190 00 Praha 9   fax: +420 2 9654 2374
Czech Republic   http://www.suse.cz/

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

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