< 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 agree as well.

 

Scott

 

From: resiprocate-devel-bounces@xxxxxxxxxxxxxxxxxxxx [mailto:resiprocate-devel-bounces@xxxxxxxxxxxxxxxxxxxx] On Behalf Of Jason Fischl
Sent: Monday, April 16, 2007 5:22 PM
To: Byron Campen
Cc: resiprocate-devel
Subject: 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

 

Project

resiprocate

New Revision

7077

Committer

nash (Nash Tsai)

Date

2007-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