[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [patch #5926] Use function pointers instead of table index
From: |
Simon Goldschmidt |
Subject: |
[lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg |
Date: |
Fri, 11 May 2007 07:53:39 +0000 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3 |
Follow-up Comment #6, patch #5926 (project lwip):
>But If others really prefer the "define" solution, we can...
No no, it's OK. Maybe it's really better because that way we only would have
to change tcpip.c.
But couldn't we move the sys_mbox_post() there, too? Because we are waiting
for the mbox in tcpip_apimsg(), I think we should post in the same file. This
would also simplify the code in api_msg.c since I think there are some cases
(maybe only if the programmer violates the api) where the mbox isn't posted
and the calling thread would wait forever.
More, I think we could replace the conn->mbox by a conn->sem if we only post
empty messages...
>But if you see something I miss, tell me..
I just wanted to know what you are talking about, seems like I missed that
discussion. But no, I don't have anyting to add :-)
I think you can check it in like that (we can move the posting to the mbox
later, it's not anything you change with your patch)
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?5926>
_______________________________________________
Nachricht geschickt von/durch Savannah
http://savannah.nongnu.org/
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Frédéric Bernon, 2007/05/10
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Frédéric Bernon, 2007/05/10
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Frédéric Bernon, 2007/05/10
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Frédéric Bernon, 2007/05/10
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Simon Goldschmidt, 2007/05/11
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Frédéric Bernon, 2007/05/11
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg,
Simon Goldschmidt <=
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Frédéric Bernon, 2007/05/11
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Simon Goldschmidt, 2007/05/11
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Frédéric Bernon, 2007/05/11
- [lwip-devel] [patch #5926] Use function pointers instead of table index in api_lib/api_msg, Simon Goldschmidt, 2007/05/11