I think it was simply done to ensure no accidental deletes outside of those classes. It is no problem to make it public though.
Greetings, On 7/23/06, Michael D. Adams <[EMAIL PROTECTED]> wrote: > Why is csSequence::RunSequenceOp::~RunSequenceOp() protected in > CS/trunk/plugins/sequence/sequence.h? I understand why it would be > protected instead of private (so children can actually call their > parent destructor), but why does it need protection instead of being > public? > > This protected status causes a problem for converting it to the new > SCF. The inheritance hierarchy looks like this: > > scfImplementation1<csSequence::StandardOperation, iSequenceOperation> > has child > csSequence::StandardOperation > has child > scfImplementationExt0<RunSequenceOp, StandardOperation> > has child > csSequence::RunSequenceOp > > And scfImplementationExt0<RunSequenceOp, StandardOperation>::DecRef() > needs to call csSequence::RunSequenceOp::~RunSequenceOp() (an error > which is flagged in the body of csSequence::AddRunSequence() which > directly calls DecRef()). > > Since scfImplementationExt0<RunSequenceOp, StandardOperation> isn't a > child (it's a parent) of csSequence::RunSequenceOp, then > scfImplementationExt0<RunSequenceOp, StandardOperation>::DecRef() > isn't allowed to do that. > > Michael D. Adams > [EMAIL PROTECTED] > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get the chance to share your > opinions on IT & business topics through brief surveys -- and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > _______________________________________________ > Crystal-main mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/crystal-main > Unsubscribe: mailto:[EMAIL PROTECTED] > -- Project Manager of Crystal Space (http://www.crystalspace3d.org) and CEL (http://cel.crystalspace3d.org) Support Crystal Space. Donate at https://sourceforge.net/donate/index.php?group_id=649 ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Crystal-main mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/crystal-main Unsubscribe: mailto:[EMAIL PROTECTED]
