Bug 93862

Summary: crash importing c++ code
Product: [Applications] umbrello Reporter: Jose N Hinckel <hinckel>
Component: generalAssignee: Umbrello Development Group <umbrello-devel>
Status: RESOLVED NOT A BUG    
Severity: crash    
Priority: NOR    
Version: 1.3.1   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: header file
header file
header file

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