Re: lib/39215: getdevmajor()/major() types don't match

[Matthias Drochner <M.Drochner%fz-juelich.de@localhost>]

The following reply was made to PR lib/39215; it has been noted by GNATS.

From: Matthias Drochner <M.Drochner%fz-juelich.de@localhost>
To: christos%zoulas.com@localhost (Christos Zoulas)
Cc: gnats-bugs%NetBSD.org@localhost, lib-bug-people%NetBSD.org@localhost,
        gnats-admin%NetBSD.org@localhost, netbsd-bugs%NetBSD.org@localhost,
        dholland%eecs.harvard.edu@localhost
Subject: Re: lib/39215: getdevmajor()/major() types don't match 
Date: Mon, 04 Aug 2008 21:08:25 +0200

 christos%zoulas.com@localhost said:
 > On Aug 1,  2:45pm, dholland-bugs%netbsd.org@localhost (David Holland) wrote:
 > | I guess it's time for devmajor_t and devminor_t?
 > Yes, and I like those names better than just major_t and minor_t since
 > they are less namespace pollutant :-) 
 
 I've just done a full build with that, to see what it takes.
 Using devmajor_t and devminor_t in the API is simple, as long
 as it stays a signed int:-)
 If I try to play with this, a lot of format string warnings
 show up because the result of major() or minor() is
 printed with "%d" at a lot of places. So typecasts are needed.
 Also the DISKUNIT() etc macros need to get explicite typecasts.
 
 That's all not too serious, but it is more work than one
 would initially expect. I'll try to come up with patches.
 
 best regards
 Matthias
 
 
 
 
 -------------------------------------------------------------------
 -------------------------------------------------------------------
 Forschungszentrum Juelich GmbH
 52425 Juelich
 
 Sitz der Gesellschaft: Juelich
 Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
 Vorsitzende des Aufsichtsrats: MinDir'in Baerbel Brumme-Bothe
 Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
 Dr. Ulrich Krafft (stellv. Vorsitzender), Prof. Dr. Harald Bolt,
 Dr. Sebastian M. Schmidt
 -------------------------------------------------------------------
 -------------------------------------------------------------------