< Previous by Date Date Index Next by Date >
  Thread Index Next in Thread >

[reSIProcate] Fwd: [reSIProcate-commit] resiprocate 7077 nash: inside resip/stack/Transaction::process method contains complex


Yikes! Can we put this in a branch somewhere so we have time to digest it? I get the distinct impression that this large of a change is going to cause a lot of headaches for a lot of people. Mind you, I can see why this sort of change might be desirable, but this seems a tad aggressive. I would also like to have a chance to look into the performance impact that this work has. I don't want to get lynched when it comes time to push 1.2 out...

Best regards,
Byron Campen

Begin forwarded message:

Date: April 16, 2007 5:07:24 AM CDT
Subject: [reSIProcate-commit] resiprocate 7077 nash: inside resip/stack/Transaction::process method contains complex

Projectresiprocate
New Revision7077
Committernash (Nash Tsai)
Date2007-04-16 05:07:24 -0500 (Mon, 16 Apr 2007)

Log

 inside resip/stack/Transaction::process method contains complex 
 logics and message ptr can be easily leaked if it's not maintained by 
 original coder, therefore I changed to SharedPtr usage and made a bunch 
 of code changed to make it compile.
 
 Also another place worth mention as TimeLimitFifo::add method could 
 result memory leak usage, and it's fixed on the way for above changes.
 
 The tests under rutil/stack/dum has passed as well.
 
 resip/stack/Security add to allow disable server authentication check
 
 
 
 

Modified:

_______________________________________________
resiprocate-commit mailing list

Attachment: smime.p7s
Description: S/MIME cryptographic signature