Re: lwIP upgrade to CVS HEAD

Simon Kallweit <[email protected]>
Newsgroups gmane.os.ecos.devel
Message-ID <[email protected]>
David Fernandez wrote:
> John Dallaway wrote:
>> There's some scope for improving the dependency info. For example,
>> CYGIMP_LWIP_MEMP_MEM_MALLOC could specify:
>>
>>    requires { CYGINT_ISO_MEMALLOC != 0 }
>>    default_value { CYGINT_ISO_MEMALLOC != 0 }
>>
> 
> Shouldn't it use "calculated_value" instead of "default_value"?

No, calculated_value would not let the user configure lwIP to use the 
internal allocator when memalloc is available. But I think there are 
scenarios where this is feasible, as it keeps all allocations for lwIP 
in a "safe place", not mixing up with the rest of the system. I was 
wondering to leave the default_value at 0, but I guess most users would 
expect lwIP to use the system memory allocator when not otherwise 
specified, so John's suggestion is fine I think.

Simon
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.