Bug 103115 - kdelibs contains invalid C++
Summary: kdelibs contains invalid C++
Status: RESOLVED REMIND
Alias: None
Product: kdelibs
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal (vote)
Target Milestone: ---
Assignee: Thiago Macieira
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-04-03 07:12 UTC by Karl Chen
Modified: 2005-04-24 17:56 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.