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