On Thu, Apr 26, 2007 at 11:38:52PM +0900, Neil Booth wrote: > > > memcpy(data + nhead, skb->head, > > >#ifdef NET_SKBUFF_DATA_USES_OFFSET > > > skb->tail); > > >#else > > > skb->tail - skb->head); > > >#endif > > > > If memcpy happens to be a macro, that is not valid C. > > It's undefined rather than invalid. GCC makes an effort to > do "what you'd expect". I think I prefer sparse breaking on things like this (so we can find such uglies and correct them.). - To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html