07.03.2014 23:06, Andreas Färber wrote:
> Am 07.03.2014 19:48, schrieb Stefan Weil:
[]
>>      if (flags & FCB_TX_IP && flags & FCB_TX_CIP) {
>> -        /* do IP4 checksum (TODO This funtion does TCP/UDP checksum but not 
>> sure
>> -         * if it also does IP4 checksum. */
>> +        /* do IP4 checksum (TODO This function does TCP/UDP checksum
>> +         * but not sure if it also does IP4 checksum. */
> 
> While rebreaking it, you could've also added the missing parenthesis,
> after the dot I guess. Apart from that,

Actually, that's a fun comment.  "do IP4 checksum, but unsure if it also does 
IP4 checksum".
Ie, do something but I'm not sure if it actually does that... ;)

Applied to -trivial, fixing both the commit comment and this parenthesis.

Thanks,

/mjt

Reply via email to