Bug 185698 - C++ class generator poor support for namespaces
Summary: C++ class generator poor support for namespaces
Status: RESOLVED FIXED
Alias: None
Product: kdevelop
Classification: Applications
Component: Language Support: CPP (old) (show other bugs)
Version: git master
Platform: Compiled Sources Unspecified
: NOR normal
Target Milestone: ---
Assignee: kdevelop-bugs-null
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-27 12:38 UTC by Joris Guisson
Modified: 2009-02-27 19:22 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch to fix this (926 bytes, text/plain)
2009-02-27 12:40 UTC, Joris Guisson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Joris Guisson 2009-02-27 12:38:42 UTC
Version:            (using KDE 4.2.0)
Installed from:    Compiled From Sources

When you generate a C++ class, and you include the nanespace name in the class name, the filenames in the wizard still include the namespace, which is not exactly what you would want.

The namespace name should be stripped of the filenames.
Comment 1 Joris Guisson 2009-02-27 12:40:10 UTC
Created attachment 31676 [details]
Patch to fix this

This patch strips away namespace names when it sees :: in the class name, tested it and it works properly.
Comment 2 Andreas Pakulat 2009-02-27 15:29:17 UTC
the right component is cpp-support, not general.
Comment 3 David Nolden 2009-02-27 19:22:13 UTC
Should be fixed now. I applied a slightly different patch, because yours does not work if there is multiple nested namespaces.