Subject: Re: Printing the driver name as part of the netdev watchdog message

From: Joe Perches <[email-masked]>
Date: Wed, 09 Jul 2008 11:20:49 -0700

On Wed, 2008-07-09 at 10:56 -0700, Arjan van de Ven wrote:
> I don't like the size_t... size_t is an abstraction to deal with file
> sizes.... but there's nothing else wrong with using an int for numbers.

<pedant_mode>

o size_t is an abstraction to deal with arbitrary sizes
o an unnecessary <= 0 test exists
o using sizeof(drivername) reduces a dependency on a magic number
o netdev_drivername calls strlcpy with implicit cast to size_t

$ grep strlcpy include/linux/string.h
size_t strlcpy(char *, const char *, size_t);

Your choice.  Cheers, Joe