[lwip-devel] [task #7490] Add return value to sys_mbox_post

[Simon Goldschmidt]

Follow-up Comment #23, task #7490 (project lwip):

to sys_arch.txt: To prevent people porting lwIP to get confused about the
'size' argument, I'd add something like 'Elements stored in mboxes are
pointers.' to the description of sys_mbox_new() so that people can calculate
the actual memory size needed. Also, 0 = max or 0 = default should be
mentioned somewhere, I think (since TCPIP_MBOX_SIZE is 0 by default).

to 1/:
I would rather include a new function (tcpip_trycallback or something), that
way, it is not linked when not used.

to 4/:
Resetting the pcb callbacks, calling tcp_close (or tcp_abort), freeing
conn->mbox, conn->recvmbox and conn should be done. This can be done in an
extra function to be combined with do_close_internal() and netconn_delete()
(draining the mboxes doesn't hurt).

About using sys_mbox_trypost in api_msg.c:recv_tcp(): Does this really work?
We should take a look at tcp_in.c to see if retransmitted data is given to the
application again on receiving a second time... after all I think tcp might
already have ACKed that data... ??

Other than that, I think the patch is fine.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/task/?7490>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.nongnu.org/