kernel dev_t elimination patch for review
rik at cronyx.ru
Fri Jun 11 15:47:39 GMT 2004
Bruce Evans wrote:
>On Fri, 11 Jun 2004, Poul-Henning Kamp wrote:
>>This patch changes "dev_t" in the kernel to "struct cdev *"
>>and is 337K of boredom like:
>> -static dev_t perfdev;
>> +static struct cdev *perfdev;
>>People who maintain cross-platform or cross-version source code in
>>the kernel, should take a close look at this patch.
>>If you don't like the way I have done it (which is mostly automated)
>>you are more than welcome to commit your own patch, all you have
>>to do is use "struct cdev *" instead of "dev_t", and you can do
>>that already now.
>This mangles the formatting in approximately 10% of cases (mainly for
>dev_t\t\t and for almost all dev_t's in structs; in structs it tends
>to misindent both the member name and the comment).
Probably this could be solved by local definition of dev_t as a struct
cdev. This would much easy
than fixing formatting in every file. After that code maintaners can
keep it or change whatever they
>freebsd-current at freebsd.org mailing list
>To unsubscribe, send any mail to "freebsd-current-unsubscribe at freebsd.org"
More information about the freebsd-current