[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [task #7136] Centralize mempool settings in separate file
From: |
Jared Grubb |
Subject: |
[lwip-devel] [task #7136] Centralize mempool settings in separate file |
Date: |
Thu, 02 Aug 2007 17:36:47 +0000 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.6) Gecko/20070725 Firefox/2.0.0.6 |
Follow-up Comment #13, task #7136 (project lwip):
I personally like seeing an explicit list of what the mempools are. But, you
do bring up a valid argument: what happens if the user does minimal
configuration. All the defaults should "just work" for the user. Also, if a
new feature is added into CVS that requires a change to the lwippools.h, but
the file is exclusively in the user's domain, then we have a maintanence
issue.
If we leave the MEMP_NUM_* options in place, then I would suggest pulling out
the standard pools into a lwip/include/memp_std.h file. Then we create two
new options:
* MEMP_USE_STD_POOLS (default 1) to give the user an ability to disable all
the standard pools and explicitly make them himself
* MEMP_USE_CUSTOM_POOLS (default 0) to indicate that he wants to add custom
pools to the list (e.g., the mem_malloc pools)
If MEMP_USE_STD_POOLS==0 or MEMP_USE_CUSTOM_POOLS==1, then the lwippools.h
file will be included.
As far as dependencies and checking, I think that would be a great idea. We
could create a header file whose sole job is to do all the sanity checks on
options that we might want.
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/task/?7136>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, (continued)
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jonathan Larmour, 2007/08/01
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Kieran Mansley, 2007/08/01
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Frédéric Bernon, 2007/08/01
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jonathan Larmour, 2007/08/01
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Frédéric Bernon, 2007/08/01
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/01
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/01
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/01
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jonathan Larmour, 2007/08/02
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Kieran Mansley, 2007/08/02
- [lwip-devel] [task #7136] Centralize mempool settings in separate file,
Jared Grubb <=
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/02
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Kieran Mansley, 2007/08/03
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/03
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/14
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Simon Goldschmidt, 2007/08/15
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/15
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/17
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Frédéric Bernon, 2007/08/20
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Jared Grubb, 2007/08/20
- [lwip-devel] [task #7136] Centralize mempool settings in separate file, Kieran Mansley, 2007/08/29