[lwip-devel] [patch #6699] Fixing a couple of compilation warnings(Paradigm C++)

[Mike Kleshov]

Follow-up Comment #6, patch #6699 (project lwip):

On closer look, it seems that the original behavior (always false) is
desireable:

if ((conn->write_msg->msg.w.len - conn->write_offset > 0xffff)) { /*
max_u16_t */
  len = 0xffff;

Apparently, the intent of the code is to limit the 'len' argument (which is
u16_t) passed to tcp_write(). When sizeof(int)==2, the amount of data is
already limited by the integer data type. So 'always false' seems appropriate
here. The easy solution is to revert the patch, then.


    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?6699>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/