git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 5/8] Load all the modules in one place and before running code.

From
Michael G. Schwern <schwern@pobox.com>
Date
Jul 27, 2012, 00:26 UTC
Message-ID
<1343348767-86446-6-git-send-email-schwern@pobox.com>
In-Reply-To
<1343348767-86446-1-git-send-email-schwern@pobox.com>
From: "Michael G. Schwern" <schwern@pobox.com>
Just makes the code easier to follow.  No functional change.
Also eliminate an unused lexical $SVN.
---
 git-svn.perl | 44 +++++++++++++++++++++-----------------------
 1 file changed, 21 insertions(+), 23 deletions(-)
diff --git a/git-svn.perl b/git-svn.perl
index 3741e2e..fc49ad6 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -10,11 +10,26 @@ use vars qw/	$AUTHOR $VERSION
 $AUTHOR = 'Eric Wong <normalperson@yhbt.net>';
 $VERSION = '@@GIT_VERSION@@';
 
+use Carp qw/croak/;
+use Digest::MD5;
+use IO::File qw//;
+use File::Basename qw/dirname basename/;
+use File::Path qw/mkpath/;
+use File::Spec;
+use File::Find;
+use Getopt::Long qw/:config gnu_getopt no_ignore_case auto_abbrev/;
+use IPC::Open3;
+use Memoize;
+
 use Git::SVN;
+use Git::SVN::Editor;
+use Git::SVN::Fetcher;
+use Git::SVN::Ra;
+use Git::SVN::Prompt;
 use Git::SVN::Log;
 use Git::SVN::Migration;
-use Git::SVN::Utils qw(fatal can_compress);
 
+use Git::SVN::Utils qw(fatal can_compress);
 use Git qw(
     git_cmd_try
     command
@@ -26,6 +41,11 @@ use Git qw(
     command_close_bidi_pipe
 );
 
+BEGIN {
+	Memoize::memoize 'Git::config';
+	Memoize::memoize 'Git::config_bool';
+}
+
 
 # From which subdir have we been invoked?
 my $cmd_dir_prefix = eval {
@@ -79,28 +99,6 @@ sub _req_svn {
 	}
 }
 
-use Carp qw/croak/;
-use Digest::MD5;
-use IO::File qw//;
-use File::Basename qw/dirname basename/;
-use File::Path qw/mkpath/;
-use File::Spec;
-use File::Find;
-use Getopt::Long qw/:config gnu_getopt no_ignore_case auto_abbrev/;
-use IPC::Open3;
-use Git::SVN::Editor qw//;
-use Git::SVN::Fetcher qw//;
-use Git::SVN::Ra qw//;
-use Git::SVN::Prompt qw//;
-use Memoize;  # core since 5.8.0, Jul 2002
-
-BEGIN {
-	Memoize::memoize 'Git::config';
-	Memoize::memoize 'Git::config_bool';
-}
-
-my ($SVN);
-
 $sha1 = qr/[a-f\d]{40}/;
 $sha1_short = qr/[a-f\d]{4,40}/;
 my ($_stdin, $_help, $_edit,
-- 
1.7.11.1
Previous: Michael G. SchwernNext: Michael G. Schwern
Message 6 of 16 in “Extract remaining classes from git-svn”
  1. Michael G. SchwernJul 27, 2012
  2. 1/8 Prepare Git::SVN::Log for extraction from git-svn.Michael G. Schwern, Jul 27, 2012
  3. 2/8 Extract Git::SVN::Log from git-svn.Michael G. Schwern, Jul 27, 2012
  4. 3/8 Prepare Git::SVN::Migration for extraction from git-svn.Michael G. Schwern, Jul 27, 2012
  5. 4/8 Extract Git::SVN::Migration from git-svn.Michael G. Schwern, Jul 27, 2012
  6. 5/8 Load all the modules in one place and before running code.Michael G. Schwern, Jul 27, 2012
  7. 6/8 Move Git::IndexInfo into its own file.Michael G. Schwern, Jul 27, 2012
  8. 7/8 Extract Git::SVN::GlobSpec from git-svn.Michael G. Schwern, Jul 27, 2012
  9. 8/8 Fix indents to match style.Michael G. Schwern, Jul 27, 2012
  10. Michael G SchwernJul 27, 2012
  11. Erik Faye-LundJul 27, 2012
  12. Michael G SchwernJul 27, 2012
  13. Eric WongJul 28, 2012
  14. Junio C HamanoJul 28, 2012
  15. Eric WongJul 28, 2012
  16. Junio C HamanoJul 28, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.