Changes between Initial Version and Version 5 of Ticket #73


Ignore:
Timestamp:
Nov 25, 2007, 11:19:55 PM (16 years ago)
Author:
mloskot
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #73

    • Property Reporter changed from mateusz@… to mloskot
    • Property Priority 5 → critical
    • Property Version 3.0.0 → svn-trunk
    • Property Milestone imported → 3.0.0
    • Property Resolution none → fixed

  • Ticket #73 – Description

    initial v5  
    1 {{{
    21Today, I identified 25 classes that have some virtual member functions but don't have a virtual destructor.
    32Those classes should have vritual destructors.
    … …  
    54The rule of thumb is:
    65
    7 "make your destructor virtual if your class has any virtual functions"
     6 '''make your destructor virtual if your class has any virtual functions'''
    87
    98It does not introduces any unnecessary overhead and does not decrease performance.
    … …  
    1312Here is a list of classes that should get virtual destructors:
    1413
     14{{{
    1515geos::geom::CoordinateSequenceFactory
    1616geos::geom::CoordinateArraySequenceFactory