<p>In <a href="https://github.com/ipxe/ipxe/pull/33#discussion-diff-22600760">src/net/udp/tftp.c</a>:</p>
<pre style='color:#555'>>      size_t len;
>    struct io_buffer *iobuf;
>    size_t blksize;
>  
> +  /* Strip initial '/' if present. If we were opened via the
> +  * URI interface, then there will be an initial '/', since a
> +  * full tftp:// URI provides no way to specify a non-absolute
> +  * path. However, many TFTP servers (particularly Windows
> +  * TFTP servers) complain about having an initial '/', and it
> +  * violates user expectations to have a '/' silently added to
> +  * the DHCP-specified filename.
> +  */
> +  path = tftp->uri->path;
</pre>
<p>how is lines 327 and 340 different from line 326?</p>

<p>I like your comment though.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/ipxe/ipxe/pull/33/files#r22600760">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AArTVKdsHr13JeWd1kEwKlexhATNEl5Rks5nfWVMgaJpZM4DPXng.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
  <div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
    <link itemprop="url" href="https://github.com/ipxe/ipxe/pull/33/files#r22600760"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>