Hi Atul! Thanks for the patch! I pushed it in revno 90 of your public branch. Totally agree that checking for 0 is not needed before a delete.
Regrads,Peter
atul wrote:
Dear all
I introduced
void TcpConnection::startTimer() and
void TcpConnection::stopTimer()
and I think this removes some duplication and makes the code more readable…. It also has the previous change ( not checking pointer for null before deleting )
– cheerio atul
