[prev in list] [next in list] [prev in thread] [next in thread] 

List:       asterisk-dev
Subject:    Re: [Asterisk-Dev] ast_xxx function..?
From:       Matt Hess <mhess () livewirenet ! com>
Date:       2005-09-30 23:55:10
Message-ID: 433DD05E.20809 () livewirenet ! com
[Download RAW message or body]


Roy Sigurd Karlsbakk wrote:
>>> I agree.. it would allow a rfc-broken device to function properly  
>>> and on that reason alone should not be done. As a counter-point to  
>>> that.. doesn't asterisk put in stuff to inter-operate better with  
>>> other almost rfc compliant devices?
>>>
>>
>> Yes, but only when it doesn't cause a performance penalty or  
>> maintenance burden.
> 
> 
> IMHO this looks like a small call to a function cleaning up a string.  
> It really can't do much of a performance
>  penalty, can it?
> 
> roy

I agree with your statement hence my initial approach of putting the 
nonce string cleaning into asterisk. But I can also see the other side 
that the approach of band-aid functions such as this is not optimal in 
that it does allow for rfc-broken devices to work better and who knows 
how much more broken they really are until full testing can be done on 
them. And to add to that.. I'd imagine if everyone started doing this 
all of these little helper functions can have an aggregate effect on 
performance.

So I guess the devs would have to decide what the balance really is as 
far as the benefit of having more devices function better when working 
with asterisk or having an overall leaner system which functions with 
rfc-compliant devices. Perhaps there's a balance to these but my 
intellect is slipping as it's been a long day for me.



> _______________________________________________
> Asterisk-Dev mailing list
> Asterisk-Dev@lists.digium.com
> http://lists.digium.com/mailman/listinfo/asterisk-dev
> To UNSUBSCRIBE or update options visit:
>   http://lists.digium.com/mailman/listinfo/asterisk-dev
> 
> 

["mhess.vcf" (text/x-vcard)]

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic