[lwip-devel] [patch #9611] tftp client

[Simon Goldschmidt]

Update of patch #9611 (project lwip):

                 Summary:  tftp client/dhcp issue => tftp client            

    _______________________________________________________

Follow-up Comment #1:

Yes, having a TFTP client integrated with the server would be welcome. But
please:
- rename or add files to reflect their contents (server vs. client)
- name functions to reflect their usage (server vs. client; tftp_init() is a
really bad example here)
- keep one patch for one issue (tftp client) and don't mix it with a bugfix
(dhcp issue) - I've change the summary for this.

On the DHCP issue, you're probably right. However, after your change, it seems
like with "parse_sname_as_options != 0", the file name could be copied
multiple times?

    _______________________________________________________

Reply to this item at:

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

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