Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 23 Apr 2014 14:05:28 +0000 (UTC)
From:      Ed Schouten <ed@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r264823 - in head/usr.bin/bsdiff: bsdiff bspatch
Message-ID:  <201404231405.s3NE5SUL064345@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ed
Date: Wed Apr 23 14:05:28 2014
New Revision: 264823
URL: http://svnweb.freebsd.org/changeset/base/264823

Log:
  Make usage printing more consistent with other tools.
  
  - Introduce a separate usage() function.
  - Don't use argv[0]. Directly name the application, as we do elsewhere.
  - Don't prepend the application name.
  - Don't print two newlines.

Modified:
  head/usr.bin/bsdiff/bsdiff/bsdiff.c
  head/usr.bin/bsdiff/bspatch/bspatch.c

Modified: head/usr.bin/bsdiff/bsdiff/bsdiff.c
==============================================================================
--- head/usr.bin/bsdiff/bsdiff/bsdiff.c	Wed Apr 23 12:52:11 2014	(r264822)
+++ head/usr.bin/bsdiff/bsdiff/bsdiff.c	Wed Apr 23 14:05:28 2014	(r264823)
@@ -196,6 +196,14 @@ static void offtout(off_t x,u_char *buf)
 	if(x<0) buf[7]|=0x80;
 }
 
+static void
+usage(void)
+{
+
+	fprintf(stderr, "usage: bsdiff oldfile newfile patchfile\n");
+	exit(1);
+}
+
 int main(int argc,char *argv[])
 {
 	int fd;
@@ -216,7 +224,8 @@ int main(int argc,char *argv[])
 	BZFILE * pfbz2;
 	int bz2err;
 
-	if(argc!=4) errx(1,"usage: %s oldfile newfile patchfile\n",argv[0]);
+	if (argc != 4)
+		usage();
 
 	/* Allocate oldsize+1 bytes instead of oldsize bytes to ensure
 		that we never try to malloc(0) and get a NULL pointer */

Modified: head/usr.bin/bsdiff/bspatch/bspatch.c
==============================================================================
--- head/usr.bin/bsdiff/bspatch/bspatch.c	Wed Apr 23 12:52:11 2014	(r264822)
+++ head/usr.bin/bsdiff/bspatch/bspatch.c	Wed Apr 23 14:05:28 2014	(r264823)
@@ -57,6 +57,14 @@ static off_t offtin(u_char *buf)
 	return y;
 }
 
+static void
+usage(void)
+{
+
+	fprintf(stderr, "usage: bspatch oldfile newfile patchfile\n");
+	exit(1);
+}
+
 int main(int argc,char * argv[])
 {
 	FILE * f, * cpf, * dpf, * epf;
@@ -72,7 +80,8 @@ int main(int argc,char * argv[])
 	off_t lenread;
 	off_t i;
 
-	if(argc!=4) errx(1,"usage: %s oldfile newfile patchfile\n",argv[0]);
+	if (argc != 4)
+		usage();
 
 	/* Open patch file */
 	if ((f = fopen(argv[3], "rb")) == NULL)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201404231405.s3NE5SUL064345>