[PATCH v2 8/8] nandwrite: use common.h "errmsg_die"

Brian Norris computersforpeace at gmail.com
Wed Dec 1 01:20:46 EST 2010


errmsg_die() should be nearly the equivalent of the error message used
here. This saves a few lines.

Also edited the error message to include the offending option and got
rid of the quotes.

Signed-off-by: Brian Norris <computersforpeace at gmail.com>
---
 nandwrite.c |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/nandwrite.c b/nandwrite.c
index 2e45136..7f459cb 100644
--- a/nandwrite.c
+++ b/nandwrite.c
@@ -212,11 +212,9 @@ static void process_options(int argc, char * const argv[])
 		}
 	}
 
-	if (mtdoffset < 0) {
-		fprintf(stderr, "Can't specify a negative device offset `%lld'\n",
-				mtdoffset);
-		exit(EXIT_FAILURE);
-	}
+	if (mtdoffset < 0)
+		errmsg_die("Can't specify negative device offset with option"
+				" -s: %lld", mtdoffset);
 
 	if (blockalign < 0)
 		errmsg_die("Can't specify negative blockalign with option -b:"
-- 
1.7.0.4





More information about the linux-mtd mailing list