user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
* [PATCH] convert: remove duplicated GetOptions() call
@ 2019-11-14  8:47 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2019-11-14  8:47 UTC (permalink / raw)
  To: meta

We only need to parse the command-line once.
---
 script/public-inbox-convert | 1 -
 1 file changed, 1 deletion(-)

diff --git a/script/public-inbox-convert b/script/public-inbox-convert
index 3182410e..9bee5e7a 100755
--- a/script/public-inbox-convert
+++ b/script/public-inbox-convert
@@ -20,7 +20,6 @@ my %opts = (
 	'--index!' => \$index,
 );
 GetOptions(%opts) or die "bad command-line args\n$usage";
-GetOptions(%opts) or die "bad command-line args\n$usage";
 my $old_dir = shift or die $usage;
 my $new_dir = shift or die $usage;
 die "$new_dir exists\n" if -d $new_dir;

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2019-11-14  8:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-14  8:47 [PATCH] convert: remove duplicated GetOptions() call Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).