On Tue, 2011-04-12 at 13:05 +0200, Karel Zak wrote: > On Tue, Apr 12, 2011 at 12:29:07PM +0200, Sami Kerola wrote: > > On Tue, Apr 12, 2011 at 12:08, Karel Zak <kzak@xxxxxxxxxx> wrote: > > >> - die(_("%s is mounted; will not make a filesystem here!")); > > >> + errx(8, _("%s is mounted; will not make a filesystem here!"), device_name); > > > > > > Please, define and use EXIT_SOMETHING instead of the magic number '8'. > > > > The disc-utils/mkfs.cramfs.c has > > > > #define MKFS_OK 0 /* No errors */ > > #define MKFS_ERROR 8 /* Operational error */ > > #define MKFS_USAGE 16 /* Usage or syntax error */ > > > > and the return value 8 seems to be somehow special for mkfs* I wonder > > if the definition of return values should be in mkfs.h > Oh, so I see these codes are kind of standard with mkfs programs... will do, thanks for pointing this out. -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html