reSIProcate

Mailing Lists

Source Code

Google Search:



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

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

I'm also not very keen on this change without doing lots of thinking first. Can I suggest that we back this out for now.

Thanks
Jason


On 4/16/07, Byron Campen <bcampen@xxxxxxxxxxxx> wrote:
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 Revision 7077
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


_______________________________________________
resiprocate-devel mailing list
resiprocate-devel@xxxxxxxxxxxxxxxxxxxx
https://list.resiprocate.org/mailman/listinfo/resiprocate-devel