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

List:       kde-devel
Subject:    Re: [PATCH] kdelibs cvs on IRIX
From:       Dominique Devriese <fritmebufstek () pandora ! be>
Date:       2002-08-30 20:19:35
[Download RAW message or body]

Jesse Barnes <jbarnes@sgi.com> writes:

> On Fri, Aug 30, 2002 at 11:47:55AM -0700, Neil Stevens wrote:
> > On Friday August 30, 2002 11:21, Jesse Barnes wrote:
> > > Index: interfaces/kmediaplayer/player.cpp
> > ...
> > > +using namespace KParts;
> > 
> > These kinds of things aren't safe, though.  This imports all kinds of 
> > symbols into the namespace.  Adding using willy-nilly is only going to 
> > casue ambiguities.
> > 
> > Or am I missing something?
> 
> Umm... I think stuff like this has been applied in the past, and I
> don't think it causes any harm (I've already resolved the namespace
> ambiguities where necessary).  And it's necessary for KDE to build on
> IRIX (which it used to do).

Have you tested for conflicts with --enable-final enabled ?
cheers
domi

-- 
I want to TAKE IT HOME and DRESS IT UP in HOT PANTS!!
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread] 

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