Bug 93862 - crash importing c++ code
Summary: crash importing c++ code
Status: RESOLVED NOT A BUG
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: 1.3.1
Platform: unspecified Linux
: NOR crash
Target Milestone: ---
Assignee: Umbrello Development Group
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-11-24 18:51 UTC by Jose N Hinckel
Modified: 2004-11-29 00:20 UTC (History)
0 users

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

Attachments
header file (2.51 KB, text/plain)
2004-11-24 18:57 UTC, Jose N Hinckel 		Details
header file (1.13 KB, text/plain)
2004-11-24 18:57 UTC, Jose N Hinckel 		Details
header file (1.30 KB, text/plain)
2004-11-24 18:58 UTC, Jose N Hinckel 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Jose N Hinckel 2004-11-24 18:51:16 UTC 
Version:           1.3.1 (using KDE 3.3.1, SuSE)
Compiler:          gcc version 3.3 20030226 (prerelease) (SuSE Linux)
OS:                Linux (i686) release 2.4.20-4GB

Application crashes catastrophycally upon importing code from working c++ header files. After inserting the classes in the left pane the application crashes and the machine rebooted without any warning.
Header files attached.
Comment 1 Jose N Hinckel 2004-11-24 18:57:18 UTC 
Created attachment 8415 [details]
header file
Comment 2 Jose N Hinckel 2004-11-24 18:57:52 UTC 
Created attachment 8416 [details]
header file
Comment 3 Jose N Hinckel 2004-11-24 18:58:27 UTC 
Created attachment 8417 [details]
header file
Comment 4 Jose N Hinckel 2004-11-24 19:02:14 UTC 
In the import code dialog the three header files are selected and passed to the application in one stroke.
Comment 5 Oliver Kellogg 2004-11-24 22:50:36 UTC 
Unable to reproduce with cvs head.
Thanks anyways, your report uncovered a different bug in 
ClassImport::addMethodParameter().
Comment 6 Martin 2004-11-26 16:41:02 UTC 
It looks like Bug #86958. According to the mentioned  bug report it shlould be fixed now. But litle test with following 

class A {
  public:
        void member() { a=1;}; //comment
};

has assured me that my proposed patch hasn't been commited to the sources.

Martin
Comment 7 Jonathan Riddell 2004-11-29 00:20:57 UTC 
The attached files import fine with Umbrello 1.3 CVS branch and CVS Head, so I have to mark this entry as invalid.

Martin's class works fine in CVS head and I've now backported the fix to 1.3 branch so it works there too.

Thanks both