On Tuesday 24 January 2012 01:19:14 Simon Glass wrote:
> On Thu, Jan 19, 2012 at 4:53 PM, Joe Hershberger wrote:
> > Rename parameter len to payload_len in NetSendUDPPacket: this name
> > more explicitly claims that it does not include the header size
> > Rename CDPHandler to CDPReceive: this is
On Thursday 19 January 2012 19:53:13 Joe Hershberger wrote:
> --- a/include/net.h
> +++ b/include/net.h
>
> -#define E802_HDR_SIZE22 /* 802 ethernet header size
> */
> + /* 802.2 + SNAP + ethernet header size */
> +#define E802_HDR_SIZE
Hi Joe,
On Thu, Jan 19, 2012 at 4:53 PM, Joe Hershberger wrote:
> Rename parameter len to payload_len in NetSendUDPPacket: this name
> more explicitly claims that it does not include the header size
> Rename CDPHandler to CDPReceive: this is not called as a handler,
> so don't name it that wa
Rename parameter len to payload_len in NetSendUDPPacket: this name
more explicitly claims that it does not include the header size
Rename CDPHandler to CDPReceive: this is not called as a handler,
so don't name it that way
Rename OPT_SIZE to OPT_FIELD_SIZE: clearer constant name and also
r
4 matches
Mail list logo