Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 8 Apr 2008 14:51:30 +1000 (EST)
From:      Bruce Evans <brde@optusnet.com.au>
To:        d@delphij.net
Cc:        delphij@FreeBSD.org, freebsd-arch@FreeBSD.org
Subject:   Re: [src] cvs commit: src/sys/ufs/ufs ufs_gjournal.c
Message-ID:  <20080408142942.B18072@delplex.bde.org>
In-Reply-To: <47FA6833.8080904@delphij.net>
References:  <20080407181250.6733810656C9@hub.freebsd.org> <47FA6833.8080904@delphij.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 7 Apr 2008, LI Xin wrote:

> Picking a random commit:
>
>>   FreeBSD src repository
>> 
>>   Modified files:
>>     sys/ufs/ufs          ufs_gjournal.c   Log:
>>   Correct function name in panic().
>>     Reported by:    kensmith
>>     Revision  Changes    Path
>>   1.3       +1 -1      src/sys/ufs/ufs/ufs_gjournal.c
>>  	if ((u_int)ino >= fs->fs_ipg * fs->fs_ncg)
>> -		panic("ffs_freefile: range: dev = %s, ino = %lu, fs = %s",
>> +		panic("ufs_gjournal_modref: range: dev = %s, ino = %lu, fs = 
>> %s",
>>  		    devtoname(dev), (u_long)ino, fs->fs_fsmnt);
>>  	if ((error = bread(devvp, cgbno, (int)fs->fs_cgsize, NOCRED, &bp))) {
>>  		brelse(bp);
>
> Is it suitable to use something like __func__ here?  I mean, will the usage 
> of __func__ encouraged practice for base/kernel code or not?

No, __func__ is an obfuscation.  It is sort of write-only.  It should
only be used when the function name is not a compile-time constant
(mainly in macros).  Its use would be an especially large style bug
in ufs, since ufs consistently doesn't use it (except in 1 recently
broken place).

The above commit also breaks the style using blind substitution which
happens to expand a line beyond 80 characters.

Bruce



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080408142942.B18072>