General types cleanup in msdosfs code only. This also helps reduce diffs with NetBSD msdosfs libraries and msdos support in makefs.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| sys/fs/msdosfs/denode.h | ||
|---|---|---|
| 106–107 ↗ | (On Diff #28570) | I noticed NetBSD still has u_long here (and elsewhere in this file) |
| sys/fs/msdosfs/denode.h | ||
|---|---|---|
| 106–107 ↗ | (On Diff #28570) | Do we want to prioritize the diff reduction or consistency with the rest of our code and style(9) ("The project is slowly moving to use the ISO/IEC 9899:1999 (``ISO C99'') unsigned integer identifiers of the form uintXX_t...")? It seems they are sometimes using uXXX_t and other times using u_XXX_t. |
| sys/fs/msdosfs/denode.h | ||
|---|---|---|
| 106–107 ↗ | (On Diff #28570) | We definitely want to use C99 fixed width types where the exist -- so we'll prefer int32_t to u_int32_t always. This doesn't apply to the other convenience typedefs though -- u_long vs unsigned long. Does NetBSD's makefs msdos support include the appropriate header(s) to define u_long already? |
| sys/fs/msdosfs/denode.h | ||
|---|---|---|
| 106–107 ↗ | (On Diff #28570) | Ah my mistake, this is simply a convenience typedef, NetBSD has the same typedefs set up in sys/types.h. I will revert these changes for the u_long and u_char typedefs. |
Comment Actions
- Removed whitespace and style changes on lines other than the ones affected in this diff
- Kept u_long, u_char, etc. as is