Bug 103115

Summary: kdelibs contains invalid C++
Product: [Unmaintained] kdelibs Reporter: Karl Chen <quarl+keyword+kdebugs.be4ca3>
Component: generalAssignee: Thiago Macieira <thiago>
Status: RESOLVED REMIND    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Debian testing   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Karl Chen 2005-04-03 07:12:55 UTC
Version:            (using KDE KDE 3.4.0)
Installed from:    Debian testing/unstable Packages
OS:                Linux

kdelibs source contains code of this form:
     
    namespace NS1 {
        struct S {
            int foo();
        };
    }
     
    using namespace NS1;
     
    namespace NS2 {
        int S::foo() { return 0; }
    }

(Specifically, KNetwork::internal.)

This is illegal C++, though gcc 3.4 allows it due to a bug:
    http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20711

Other compilers such as Intel C++ reject this.
Comment 1 Stephan Kulow 2005-04-03 12:23:18 UTC
you're quicker if you're simply adding a patch to make it compile instead of trying to give us a lecture
Comment 2 Thiago Macieira 2005-04-03 16:41:57 UTC
It would be easier if you said what files had this invalid construct. I thought I had got rid of them a while ago.
Comment 3 Thiago Macieira 2005-04-24 17:56:59 UTC
No reply, closing. I can't find that construct anywhere. Please point out where it is.