Bug 105388 - auto-generated accessor methods do not contain 'ClassName::' in method definition
Summary: auto-generated accessor methods do not contain 'ClassName::' in method defini...
Status: RESOLVED FIXED
Alias: None
Product: kdevelop
Classification: Applications
Component: Language Support: CPP (old) (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: kdevelop-bugs-null
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-10 07:25 UTC by Ernestas
Modified: 2006-09-06 22:26 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
fix for non-inline C++ getter/setter generation (860 bytes, patch)
2006-08-15 22:17 UTC, Ernestas Vaiciukevičius 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Ernestas 2005-05-10 07:25:27 UTC 
Version:           3.2.0 (using KDE KDE 3.4.0)
Installed from:    Debian testing/unstable Packages
Compiler:          gcc 3.3 
OS:                Linux

Auto-generated accessor methods do not contain 'ClassName::' in method definition (.cpp file).

How to reproduce:
select an attribute from class view, from context menu choose 'Generate get/set methods', uncheck 'inline' checkboxes and press ok.
Comment 1 Alberto González 2005-06-24 01:01:41 UTC 
Kdevelop in Ubuntu Hoary has the bug too: the generated code is

type accessor() {
   return val;
}

and should be

type ClassName::accessor() {
   return val;
}
Comment 2 Ernestas Vaiciukevičius 2006-08-15 22:17:01 UTC 
Created attachment 17383 [details]
fix for non-inline C++ getter/setter generation

I think It fixes getter/setter generation (at least for me)..
Comment 3 Christian Schneider 2006-08-29 14:41:48 UTC 
The same problem with SVN revision 578441 in branch svn://anonsvn.kde.org/home/kde/branches/kdevelop/3.4
Comment 4 Christian Schneider 2006-08-29 14:59:37 UTC 
The patch works fine for me too, thank you!
Comment 5 Jens Dagerbo 2006-09-06 22:26:08 UTC 
Confirmed. Patch works. Committed revision 581603. Thanks!