Thread (3 messages) flat view 3 messages, 2 authors, 2016-06-15

Re: [PATCH] Git.pm: config_boolean() -> config_bool()

From: Petr Baudis <hidden>
Date: 2016-06-15 22:43:09

Possibly related (same subject, not in this thread)

On Wed, May 09, 2007 at 12:49:41PM CEST, Petr Baudis wrote:
quoted hunk ↗ jump to hunk
@@ -526,14 +528,16 @@ This currently wraps command('config') so it is not so fast.
 
 =cut
 
-sub config_boolean {
+sub config_bool {
 	my ($self, $var) = @_;
 	$self->repo_path()
 		or throw Error::Simple("not a repository");
 
 	try {
-		return $self->command_oneline('config', '--bool', '--get',
-					      $var);
+		my $val = $self->command_oneline('config', '--bool', '--get',
+					      $val);
                                               ^^^

Sorry, should be $var. Looks like I forgot to make install or something
when "testing" the patch, also judging by the error messages repo.or.cz
spewed furiously in the last update round.

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
Ever try. Ever fail. No matter. // Try again. Fail again. Fail better.
		-- Samuel Beckett
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help