
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">

<HTML>

<HEAD>

<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1">

<META NAME="Generator" CONTENT="MS Exchange Server version 6.5.7650.28">

<TITLE>RE: [PATCH v2 5/11] Implementation of Data path of the communication protocol</TITLE>

</HEAD>

<BODY>

<!-- Converted from text/plain format -->


<P><FONT SIZE=2>> While importing these patches, I got several "Space in indent is<BR>

> followed by a tab." errors.  For example, the line<BR>

<BR>

>> +                       __constant_cpu_to_be16(ETH_P_8021Q))) {<BR>

<BR>

I did test my patches with git-apply --whitespace=error-all,<BR>

before mailing them but did not see any error.<BR>

I am using GIT 1.3, may be I should upgrade ?<BR>

<BR>

It will be great if you can share the options you use<BR>

for applying patches. That way I can test them properly before<BR>

mailing out to you.<BR>

<BR>

> which also leads to the comment that there's no reason for<BR>

> __constant_cpu_to_be16() here -- just use cpu_to_be16 and let the<BR>

> compiler do the optimization.  (the __constant form is only needed in<BR>

> places where the function call is a syntax error)<BR>

<BR>

Thanks for explaining that. I will convert these back to cpu_to_beXX<BR>

and also work on the other naming related suggestions you had.<BR>

<BR>

Regards,<BR>

Ram<BR>

<BR>

</FONT>

</P>


</BODY>

</HTML>