-
-
usr/src/cmd/fdisk/fdisk.c (Diff revision 1) Why is it returns silently instead of exiting with an error message?
-
Not for this issue.
Duplicated endian conversion macros. -
usr/src/cmd/fdisk/fdisk.c (Diff revision 1) These endian conversion macros aren't needed, instead use
LE_*
macros from<sys/byteorder.h>
(wich is already included). -
-
Change Summary:
Change suggested by Gergő Mihály Doma
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+28 -27) |
-
C-style issues.
-
-
usr/src/cmd/fdisk/fdisk.c (Diff revision 2) Missing braces.
From 'C Style and Coding Standards for SunOS' :In either case, if one arm of an if-else statement contains braces, all arms should contain braces.
-
-
Change Summary:
cstyle cleanup
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+47 -45) |
-
Man page issues.
-
usr/src/man/man1m/fdisk.1m (Diff revision 3) The lines of licensing information are too long (it seems like this is true for other man pages too). Exception: "usr/src/man/man1has/ex.1has" contains a better formatted CDDL information. Is it permitted to replace this with that can be found in "/usr/src/prototypes/prototype.man", or it should be preserved as-is (for ex.: with the same dead url)?
-
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 4 (+48 -46) |
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 5 (+48 -46) |