View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0003304 | Spring engine | General | public | 2012-10-15 23:51 | 2012-10-16 20:42 | ||||
Reporter | abma | ||||||||
Assigned To | abma | ||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||
Status | resolved | Resolution | fixed | ||||||
Product Version | 91.0.1+git | ||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0003304: compile warnings: "deleting object of polymorphic class type... which has non-virtual destructor" with gcc 4.7 | ||||||||
Description | warning: deleting object of polymorphic class type ‘CAirBaseHandler’ which has non-virtual destructor might cause undefined behaviour [-Wdelete-non-virtual-dtor] | ||||||||
Steps To Reproduce | see validation-slave | ||||||||
Additional Information | output of make -j4 >/dev/null http://paste.springfiles.com/view/raw/922badfe | ||||||||
Tags | No tags attached. | ||||||||
Checked infolog.txt for Errors | |||||||||
Attached Files |
|
![]() |
|
Kloot (developer) 2012-10-16 01:19 |
Odd, none of the classes it complains about are actually polymorphic. The only thing they have in common is that all of them derive from boost::noncopyable... |
abma (administrator) 2012-10-16 02:43 Last edited: 2012-10-16 02:43 |
only common thing i see is creg, as it seems to add/change/...? some virtual functions? |
Kloot (developer) 2012-10-16 15:04 |
That's probably it, see etherpad. :) |
abma (administrator) 2012-10-16 17:38 |
for the docs: replace CR_DECLARE by CR_DECLARE_STRUCT in all classes which generate this warning (see comments in creg.h about the difference) |
abma (administrator) 2012-10-16 17:39 |
if nobody already did it, i'll fix that |
abma (administrator) 2012-10-16 20:42 |
https://github.com/spring/spring/commit/a9f97665f79a5046dbd0eccfccdac4a6cde6c6a5 |
![]() |
|||
Date Modified | Username | Field | Change |
---|---|---|---|
2012-10-15 23:51 | abma | New Issue | |
2012-10-16 01:19 | Kloot | Note Added: 0009258 | |
2012-10-16 02:43 | abma | Note Added: 0009261 | |
2012-10-16 02:43 | abma | Note Edited: 0009261 | View Revisions |
2012-10-16 02:43 | abma | Note Edited: 0009261 | View Revisions |
2012-10-16 15:04 | Kloot | Note Added: 0009263 | |
2012-10-16 17:38 | abma | Note Added: 0009264 | |
2012-10-16 17:39 | abma | Assigned To | => abma |
2012-10-16 17:39 | abma | Status | new => assigned |
2012-10-16 17:39 | abma | Note Added: 0009265 | |
2012-10-16 20:42 | abma | Note Added: 0009267 | |
2012-10-16 20:42 | abma | Status | assigned => resolved |
2012-10-16 20:42 | abma | Resolution | open => fixed |